[Cashtab] currency.prefixes & currency.tokenPrefixes converted to strings
ClosedPublic
Actions

Authored by kieran709 on Jul 12 2022, 19:09.

Details

Reviewers

bytesofman

Group Reviewers

Restricted Project

Commits

rABC1b9a4de8a9b5: [Cashtab] currency.prefixes & currency.tokenPrefixes converted to strings

Summary

Related to T2100. currency.prefixes and currency.tokenPrefixes have been converted from arrays to strings.

Test Plan

cd web/cashtab
grep -r 'currency.prefixes' src/ should return nothing
grep -r 'currency.tokenPrefixes' should return nothing.

As neither prefix is used in the app, no other testing should be required as changing the type will have no impact on existing code.

Diff Detail

Repository

rABC Bitcoin ABC

Branch

arcpatch-D11741

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 20100
Build 39887: Build Diff	cashtab-tests
Build 39886: arc lint + arc unit

Event Timeline

kieran709 created this revision.Jul 12 2022, 19:09

Herald added a reviewer: Restricted Project. · View Herald TranscriptJul 12 2022, 19:09

kieran709 requested review of this revision.Jul 12 2022, 19:09

Harbormaster completed remote builds in B19624: Diff 34346.Jul 12 2022, 19:13

If these entries are no longer used anywhere in the app, please remove them from the currency object

This revision now requires changes to proceed.Jul 19 2022, 21:20

I noticed that in ScanQrCode there is a reference on line 72 to currency.tokenPrefix, which does not exist in the currency object, so it will not be a problem to remove them altogether, but we may need another task / diff to fix what is happening in ScanQrCode.

In D11741#265200, @kieran709 wrote:

I noticed that in ScanQrCode there is a reference on line 72 to currency.tokenPrefix, which does not exist in the currency object, so it will not be a problem to remove them altogether, but we may need another task / diff to fix what is happening in ScanQrCode.

Please create a task to correct this and assign it to yourself.

What's the status on this?

Rebased to master, removed unused currency.prefixes key/value from currency obj, changed currency.tokenPrefixes to a string.

Harbormaster completed remote builds in B20100: Diff 35074.Sep 21 2022, 13:38

grep -r 'currency.tokenPrefixes' should return nothing.

This is still being used in ScanQRCode.js

Reviewing that document, it's only used to trigger an analytics event, which we aren't using for anything.

So, please remove it from that file, then also remove it from Ticker.js

This revision now requires changes to proceed.Sep 21 2022, 23:20

Responding to review feedback, removed GA event from ScanQrCode, removed imports for currency obj & GA. Removed tokenprefixes key/value pair from currency obj.

Harbormaster completed remote builds in B20135: Diff 35122.Sep 22 2022, 19:38

bytesofman accepted this revision.Sep 22 2022, 20:19

This revision is now accepted and ready to land.Sep 22 2022, 20:19

Closed by commit rABC1b9a4de8a9b5: [Cashtab] currency.prefixes & currency.tokenPrefixes converted to strings (authored by kieran709). · Explain WhySep 26 2022, 18:41

This revision was automatically updated to reflect the committed changes.

kieran709 added a commit: rABC1b9a4de8a9b5: [Cashtab] currency.prefixes & currency.tokenPrefixes converted to strings.