[Cashtab] [wallet mgmt improvements p3] Get all wallet context variables in the same call
ClosedPublic
Actions

Authored by bytesofman on Dec 10 2022, 18:11.

Details

Reviewers

emack

Group Reviewers

Restricted Project

Commits

rABC6751af1c5cfc: [Cashtab] [wallet mgmt improvements p3] Get all wallet context variables in the…

Summary

T2822

Depends on D12842

Artifact here. As additional wallet context variables were needed, they were added in a separate call. Should just be one call.

Test Plan

Confirm the same variables are being pulled in as before.
npm start
Navigate to settings page
Change active wallet & no issues

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bytesofman created this revision.Dec 10 2022, 18:11

Herald added a reviewer: Restricted Project. · View Herald TranscriptDec 10 2022, 18:11

bytesofman requested review of this revision.Dec 10 2022, 18:11

Harbormaster completed remote builds in B21511: Diff 37183.Dec 10 2022, 18:13

bytesofman added a child revision: D12844: [Cashtab] [wallet mgmt improvements p4] Add UI locking to useWallet.js functions that change local storage.Dec 10 2022, 18:26

emack accepted this revision.Dec 11 2022, 03:27

This revision is now accepted and ready to land.Dec 11 2022, 03:27

Closed by commit rABC6751af1c5cfc: [Cashtab] [wallet mgmt improvements p3] Get all wallet context variables in the… (authored by bytesofman). · Explain WhyDec 11 2022, 05:52

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABC6751af1c5cfc: [Cashtab] [wallet mgmt improvements p3] Get all wallet context variables in the….

Revision Contents
Changeset List

Path

Size

web/

cashtab/

src/

components/

Configure/

Configure.js

11 lines

Diff 37197

View Options

web/cashtab/src/components/Configure/Configure.js

[Cashtab] [wallet mgmt improvements p3] Get all wallet context variables in the same callClosedPublicActions