Paths

Table of Contentst

Differential D12209

[Cashtab] [pub key fetch from chronik p2] Deprecate bch-api
ClosedPublic
Actions

Authored by bytesofman on Oct 11 2022, 16:09.

Details

Reviewers

emack

Group Reviewers

Restricted Project

Commits

rABC5f643846b1d5: [Cashtab] [pub key fetch from chronik p2] Deprecate bch-api

Summary

T2730

Depends on D12180

Remove bch-api calls and debug logging. Add more specific error handling for edge case failures.

Test Plan

npm start
Send an encrypted msg to an address with outgoing txs, no issues
Send an encrypted msg to an address without outgoing txs, fails with expected error msg

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bytesofman created this revision.Oct 11 2022, 16:09

Herald added a reviewer: Restricted Project. · View Herald TranscriptOct 11 2022, 16:09

bytesofman requested review of this revision.Oct 11 2022, 16:09

Harbormaster completed remote builds in B20444: Diff 35586.Oct 11 2022, 16:11

emack accepted this revision.Oct 12 2022, 01:14

This revision is now accepted and ready to land.Oct 12 2022, 01:14

Closed by commit rABC5f643846b1d5: [Cashtab] [pub key fetch from chronik p2] Deprecate bch-api (authored by bytesofman). · Explain WhyOct 12 2022, 03:50

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABC5f643846b1d5: [Cashtab] [pub key fetch from chronik p2] Deprecate bch-api.

Revision Contents
Changeset List

Path

Size

web/

cashtab/

src/

hooks/

useBCH.js

39 lines

utils/

cashMethods.js

15 lines

Diff 35597

View Options

web/cashtab/src/hooks/useBCH.js