[Cashtab] [Cashtab] [final bch-api deprecation p3] Initialize ChronikClient as a state variable
ClosedPublic
Actions

Authored by bytesofman on Dec 7 2022, 14:43.

Details

Reviewers

emack

Group Reviewers

Restricted Project

Commits

rABCa5abb85dbbe5: [Cashtab] [Cashtab] [final bch-api deprecation p3] Initialize ChronikClient as…

Summary

T2730, T2735

Depends on D12788

Move ChronikClient to a state variable

Test Plan

npm start
In dev log, confirm chronik still subscribes to websockets as expected

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bytesofman created this revision.Dec 7 2022, 14:43

Herald added a reviewer: Restricted Project. · View Herald TranscriptDec 7 2022, 14:43

bytesofman requested review of this revision.Dec 7 2022, 14:43

bytesofman edited the summary of this revision. (Show Details)Dec 7 2022, 14:43

Harbormaster completed remote builds in B21421: Diff 37053.Dec 7 2022, 14:46

bytesofman added a child revision: D12791: [Cashtab] [Cashtab] [final bch-api deprecation p4] Refactor tryNextApi function to try next chronik instance.Dec 7 2022, 14:52

emack accepted this revision.Dec 8 2022, 01:17

This revision is now accepted and ready to land.Dec 8 2022, 01:17

Closed by commit rABCa5abb85dbbe5: [Cashtab] [Cashtab] [final bch-api deprecation p3] Initialize ChronikClient as… (authored by bytesofman). · Explain WhyDec 8 2022, 03:53

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABCa5abb85dbbe5: [Cashtab] [Cashtab] [final bch-api deprecation p3] Initialize ChronikClient as….

Revision Contents
Changeset List

Path

Size

web/

cashtab/

src/

hooks/

useWallet.js

3 lines

Diff 37083

View Options

web/cashtab/src/hooks/useWallet.js

[Cashtab] [Cashtab] [final bch-api deprecation p3] Initialize ChronikClient as a state variableClosedPublicActions