Differential D11242

[Cashtab] Moved param alert above send button
ClosedPublic
Actions

Authored by kieran709 on Mar 22 2022, 15:32.

Details

Reviewers

bytesofman

Group Reviewers

Restricted Project

Commits

rABC54ee1d8ff8d4: [Cashtab] Moved param alert above send button

Summary

Related to task T2328. Param alert message on send tab has been moved above the 'Send' button.

Test Plan

cd web/cashtab && npm start
navigate to send tab
input an address that includes an amount param
observe that the param alert message appears above the send button

Diff Detail

Repository

rABC Bitcoin ABC

Branch

move-param-alert-and-add-padding

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 18598
Build 36987: Build Diff	cashtab-tests
Build 36986: arc lint + arc unit

Event Timeline

kieran709 created this revision.Mar 22 2022, 15:32

Herald added a reviewer: Restricted Project. · View Herald TranscriptMar 22 2022, 15:32

kieran709 requested review of this revision.Mar 22 2022, 15:32

Harbormaster completed remote builds in B18598: Diff 32890.Mar 22 2022, 15:43

bytesofman accepted this revision.Mar 22 2022, 18:20

This revision is now accepted and ready to land.Mar 22 2022, 18:20

Closed by commit rABC54ee1d8ff8d4: [Cashtab] Moved param alert above send button (authored by kieran709). · Explain WhyMar 22 2022, 18:37

This revision was automatically updated to reflect the committed changes.

kieran709 added a commit: rABC54ee1d8ff8d4: [Cashtab] Moved param alert above send button.

Revision Contents
Changeset List

Path

Size

web/

cashtab/

src/

components/

Send/

Send.js

13 lines

Diff 32890

View Options

web/cashtab/src/components/Send/Send.js

[Cashtab] Moved param alert above send buttonClosedPublicActions