Home's "Send" and "Receive", src/popup/views/home.js, with state.selectedToken still set.
What happens
Found while fixing #751, and run with tests on #752. Three routes reach Home with the token selected:
reopening the popup after AddressToken's "Send";
the Settings gear on AddressToken, then adding a wallet;
the Settings gear on AddressToken, then deleting another wallet, then "Back".
Home's "Send" then opens with the dropdown on ETH and reviews the selected token, whatever the dropdown shows. ConfirmTx does name the token. Home's "Receive" reads the same field, so it can show that token's warning.
What should happen
The token the screen shows is the token that is sent.
Severity: major
Definition of done
Home's "Send" and "Receive" clear state.selectedToken when they open, and Send's dropdown starts on ETH.
A Jest test for each of the three routes, failing on next: choosing ETH on Send reviews ETH.
A Jest test that Home's "Receive" on such a route shows no token warning, failing on next.
Opening Send from AddressToken still sends that token.
Model: opus-5-5
## Where
Home's "Send" and "Receive", `src/popup/views/home.js`, with `state.selectedToken` still set.
## What happens
Found while fixing https://git.eeqj.de/sneak/AutistMask/issues/751, and run with tests on https://git.eeqj.de/sneak/AutistMask/pulls/752. Three routes reach Home with the token selected:
- reopening the popup after AddressToken's "Send";
- the Settings gear on AddressToken, then adding a wallet;
- the Settings gear on AddressToken, then deleting another wallet, then "Back".
Home's "Send" then opens with the dropdown on ETH and reviews the selected token, whatever the dropdown shows. ConfirmTx does name the token. Home's "Receive" reads the same field, so it can show that token's warning.
## What should happen
The token the screen shows is the token that is sent.
Severity: major
## Definition of done
- [ ] Home's "Send" and "Receive" clear `state.selectedToken` when they open, and Send's dropdown starts on ETH.
- [ ] A Jest test for each of the three routes, failing on `next`: choosing ETH on Send reviews ETH.
- [ ] A Jest test that Home's "Receive" on such a route shows no token warning, failing on `next`.
- [ ] Opening Send from AddressToken still sends that token.
Model: opus-5-5
clawbot
self-assigned this 2026-11-03 15:27:57 +01:00
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Where
Home's "Send" and "Receive",
src/popup/views/home.js, withstate.selectedTokenstill set.What happens
Found while fixing #751, and run with tests on #752. Three routes reach Home with the token selected:
Home's "Send" then opens with the dropdown on ETH and reviews the selected token, whatever the dropdown shows. ConfirmTx does name the token. Home's "Receive" reads the same field, so it can show that token's warning.
What should happen
The token the screen shows is the token that is sent.
Severity: major
Definition of done
state.selectedTokenwhen they open, and Send's dropdown starts on ETH.next: choosing ETH on Send reviews ETH.next.Model: opus-5-5