PASS: the change closes sneak/AutistMask#522, and the README now describes how ConfirmTx, TxApproval and the wait, success and error screens show an amount exactly…
README's Display Consistency sentence now limits the base-units rule to the approval screen and the screens carrying its amount, and says ConfirmTx and the wait, success and error screens after it…
Review: FAIL (needs-rework)
tests/uniswapRecipient.test.js,the command each release runs for a command type. No test covers0x14onUniversalRouterV1or `UniversalRouterV1_2_V2Su…
FAIL (needs-rework)
README.md, Display Consistency, the sentence "The floor applies only where the token's scale is known": this PR sends the ConfirmTx amount of a token whose scale is…
The README's ConfirmTx Amount line and the comment above shownAmount() now say that a token whose scale is unknown, which cannot be sent, has its amount cut as typed (.5 reads .5000); the…
FAIL (needs-rework)
README.md, ConfirmTx's Amount line, againstshownAmount()insrc/popup/views/confirmTx.js: the README now says the amount reads the same "however Send's amount…
PASS: a balance check whose input does not decode is now listed as Balance Check (could not be decoded) and sets no minimum, as the README now says.
Judgement call: the PR body, at 256 words,…
- A
BALANCE_CHECK_ERC20whose input does not decode is now listed asBalance Check (could not be decoded), README's Transaction Decoding section says so, and a test covers an owner word with…
ConfirmTx now formats the amount parsed at the scale it is signed with and written back as plain decimal text, so .5 and 00.5 read 0.5000 on ConfirmTx and the wait, success and error…