Page MenuHomePhabricator

[Cashtab] Fix NaN render bug in paybutton links without specified amount
ClosedPublic

Authored by bytesofman on Sep 12 2023, 23:56.

Details

Summary

T3278

Prevent 'NaN' from being rendered for extension open by link with amount unspecified

Test Plan

npm test

npm run extension
load in browser
test donation button at bottom of https://coin.dance/ and opens without NaN msg

Diff Detail

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

Event Timeline

cashtab/extension/public/manifest.json
6 ↗(On Diff #42165)

note that this bug is only in the extension.

cashtab/src/components/Send/Send.js
971 ↗(On Diff #42165)

without this, the user may click "send" before entering a value, and then needs to wait for a validation error

cashtab/src/components/Send/__tests__/__snapshots__/Send.test.js.snap
1 ↗(On Diff #42165)

Note: these used to be hidden with @generated, but they should be included in review as their changes should be consistent with changes in the diff

This revision is now accepted and ready to land.Sep 13 2023, 00:27