Page MenuHomePhabricator
Differential D12654

[Cashtab] [Address simplification p4/4] Remove unused toLegacyToken function
ClosedPublic
Actions

Authored by bytesofman on Nov 27 2022, 13:41.

Details

Reviewers
emack
Group Reviewers
Restricted Project
Commits
rABC4ec02b117a3e: [Cashtab] [Address simplification p4/4] Remove unused toLegacyToken function
Summary

T2730

Depends on D12653

This function was only used in SendToken.js, where it was removed in part 1. So, delete the function and its associated unit tests.

Test Plan

npm test
grep -r toLegacyToken src/ to confirm it is nowhere in Cashtab

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bytesofman created this revision.Nov 27 2022, 13:41
Herald added a reviewer: Restricted Project. · View Herald TranscriptNov 27 2022, 13:41
bytesofman requested review of this revision.Nov 27 2022, 13:41
Harbormaster completed remote builds in B21184: Diff 36703.Nov 27 2022, 13:44
bytesofman retitled this revision from [Cashtab] [Address simplification p4] Remove unused toLegacyToken function to [Cashtab] [Address simplification p4/4] Remove unused toLegacyToken function.Nov 27 2022, 13:45
emack accepted this revision.Nov 28 2022, 02:11
This revision is now accepted and ready to land.Nov 28 2022, 02:11
Closed by commit rABC4ec02b117a3e: [Cashtab] [Address simplification p4/4] Remove unused toLegacyToken function (authored by bytesofman). · Explain WhyNov 28 2022, 04:27
This revision was automatically updated to reflect the committed changes.
bytesofman added a commit: rABC4ec02b117a3e: [Cashtab] [Address simplification p4/4] Remove unused toLegacyToken function.

Revision Contents
Changeset List

PathSize
web/
cashtab/
src/
utils/
__tests__/
11 lines
23 lines

Diff 36715

View Options

web/cashtab/src/utils/__tests__/cashMethods.test.js

Loading...
View Options

web/cashtab/src/utils/cashMethods.js

Loading...