Paths

Table of Contentst

Differential D15970

[Cashtab] After importing a wallet, automatically close the import wallet modal
ClosedPublic
Actions

Authored by bytesofman on Apr 12 2024, 23:34.

Details

Reviewers

emack

Group Reviewers

Restricted Project

Commits

rABC22c8b896b373: [Cashtab] After importing a wallet, automatically close the import wallet modal

Summary

T3531

Came across this using the app. The import wallet modal should close after we successfully import a new wallet.

Update behavior and tests to confirm behavior.

Test Plan

npm test

Diff Detail

Repository

rABC Bitcoin ABC

Branch

close-import-modal

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 28540
Build 56621: Build Diff	cashtab-tests
Build 56620: arc lint + arc unit

Event Timeline

bytesofman created this revision.Apr 12 2024, 23:34

Herald added a reviewer: Restricted Project. · View Herald TranscriptApr 12 2024, 23:34

bytesofman published this revision for review.Apr 12 2024, 23:36

bytesofman edited the summary of this revision. (Show Details)

Harbormaster completed remote builds in B28540: Diff 47122.Apr 12 2024, 23:37

emack accepted this revision.Apr 13 2024, 10:12

This revision is now accepted and ready to land.Apr 13 2024, 10:12

Closed by commit rABC22c8b896b373: [Cashtab] After importing a wallet, automatically close the import wallet modal (authored by bytesofman). · Explain WhyApr 13 2024, 13:35

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABC22c8b896b373: [Cashtab] After importing a wallet, automatically close the import wallet modal.

Revision Contents
Changeset List

Path

Size

cashtab/

src/

components/

Wallets/

__tests__/

index.test.js

27 lines

index.js

3 lines

Diff 47122

View Options

cashtab/src/components/Wallets/tests/index.test.js

View Options

cashtab/src/components/Wallets/index.js

[Cashtab] After importing a wallet, automatically close the import wallet modalClosedPublicActions