Page MenuHomePhabricator

[Cashtab] Rename WalletInfo Component in Receive.js
ClosedPublic

Authored by kieran709 on Oct 25 2022, 17:42.

Details

Reviewers
bytesofman
Group Reviewers
Restricted Project
Commits
rABC23af147f7dfe: [Cashtab] Rename WalletInfo Component in Receive.js
Summary

Related to T2742. The WalletInfo component is misnamed, has been renamed to ReceiveWithWalletPresent to more closely reflect what this component is.

Test Plan

cd web/cashtab && npm start
navigate to the receive page, observe nothing has changed
in a private browser, navigate to localhost:3000/#/receive
navigate to the receive page, observe that nothing has changed

Diff Detail

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

Event Timeline

bytesofman added inline comments.
web/cashtab/src/components/Receive/Receive.js
55 ↗(On Diff #35997)

Just call it 'Receive'

This revision now requires changes to proceed.Oct 25 2022, 17:56
bytesofman added inline comments.
web/cashtab/src/components/Receive/Receive.js
55 ↗(On Diff #35997)

oh nm I see this would conflict with the overall export.

This is fine

This revision is now accepted and ready to land.Oct 25 2022, 17:56