Differential D12849

[Cashtab] [wallet mgmt improvements p8] Remove or improve debug logging
ClosedPublic
Actions

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

Details

Reviewers

emack

Group Reviewers

Restricted Project

Commits

rABC70f175400ec0: [Cashtab] [wallet mgmt improvements p8] Remove or improve debug logging

Summary

T2822

Depends on D12848

Remove debug logging used in previous diffs on this stack.

Test Plan

Confirm that only console.log statements have been edited.

Diff Detail

Repository

rABC Bitcoin ABC

Branch

stack-to-improve-wallet-mgmt

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 21517
Build 42681: Build Diff	cashtab-tests
Build 42680: arc lint + arc unit

Event Timeline

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

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

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

bytesofman added a child revision: D12850: [Cashtab] [wallet mgmt improvements p9] Patch async function call bug in useEffect.Dec 10 2022, 18:56

Harbormaster completed remote builds in B21517: Diff 37189.Dec 10 2022, 18:57

emack accepted this revision.Dec 11 2022, 05:06

This revision is now accepted and ready to land.Dec 11 2022, 05:06

Closed by commit rABC70f175400ec0: [Cashtab] [wallet mgmt improvements p8] Remove or improve debug logging (authored by bytesofman). · Explain WhyDec 11 2022, 05:52

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABC70f175400ec0: [Cashtab] [wallet mgmt improvements p8] Remove or improve debug logging.

Revision Contents
Changeset List

Path

Size

web/

cashtab/

src/

components/

Configure/

Configure.js

5 lines

hooks/

useWallet.js

40 lines

Diff 37189

View Options

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