Page MenuHomePhabricator

[Cashtab] Stop redirecting to /wallet route from cashtab home
ClosedPublic

Authored by bytesofman on Apr 15 2024, 21:51.

Details

Summary

T3461

The home page isn't really the wallet page. Right now, it shows tx history and balance summary. May change in the future.

Remove hardcoded redirect that always took cashtab.com to /wallet.

Stop using selectedKey var, which is taking location.pathname without the '/'. Instead, use `location.pathname' directly.

Maintain support for cashtab.com/#/wallet in case someone clicks a link there or navigates directly there.

Test Plan

npm test

npm start, navigate to http://localhost:3000/#/wallet, see normal home page and home icon is eCash blue

Diff Detail

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

Event Timeline

highlight home screen if user navigates there using legacy route

bytesofman retitled this revision from [Cashtab] Stop adding wallet route to home to [Cashtab] Stop redirecting to /wallet route from cashtab home.
This revision is now accepted and ready to land.Apr 16 2024, 01:18