Page MenuHomePhabricator

[Cashtab] Remove all unused imports in chronik.test.js
ClosedPublic

Authored by kieran709 on Nov 1 2022, 17:07.

Details

Reviewers
bytesofman
Group Reviewers
Restricted Project
Commits
rABCc7aefa86d285: [Cashtab] Remove all unused imports in chronik.test.js
Summary

Related to T2713. Unused BigNumber and currency imports have been removed from chronik.test.js

Test Plan

navigate to web/cashtab/src/utils/__tests__/chronik.test.js
observe that the 'BigNumber' import is no longer present on line 2
observe that the 'currency' import is no longer present on line 1

Diff Detail

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

Event Timeline

bytesofman requested changes to this revision.Nov 1 2022, 19:10

Look's like currency is also not used -- please remove all unused imports from chronik.test.js in this diff. thanks!

This revision now requires changes to proceed.Nov 1 2022, 19:10
kieran709 retitled this revision from [Cashtab] Remove unused BigNumber import chronik.test.js to [Cashtab] Remove all unused imports in chronik.test.js.Nov 1 2022, 20:55
kieran709 edited the summary of this revision. (Show Details)
kieran709 edited the test plan for this revision. (Show Details)
kieran709 edited the test plan for this revision. (Show Details)

Responding to review feedback, removed unused currency import in chronik.test.js and updated diff details to reflect this change.

This revision is now accepted and ready to land.Nov 1 2022, 22:24