Differential D11522

[Cashtab] [Chronik] Chronikfy existing BCH.Blockchain.getBlockCount() calls
AbandonedPublic
Actions

Authored by emack on May 26 2022, 02:09.

Details

Reviewers

bytesofman

Group Reviewers

Restricted Project

Summary

Swap out existing bch-api calls retrieving the latest block height with the chronik client.

Chronik client is instantiated in useWallet and then referenced elsewhere throughout the app for efficiency.

Dependencies: chronik-client@0.6.0, long, protobufjs

Test Plan

npm start
note the debug statement in Airdrop.calculateXecAirdrop() to display the latest block height upon airdrop execution and ensure it matches the latest block shown on explorer.be.cash
ensure no regression errors with airdrop calculation

Diff Detail

Repository

rABC Bitcoin ABC

Branch

chronikBlockheight

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 19156
Build 38060: Build Diff	cashtab-tests
Build 38059: arc lint + arc unit

Event Timeline

emack created this revision.May 26 2022, 02:09

Herald added a reviewer: Restricted Project. · View Herald TranscriptMay 26 2022, 02:09

emack requested review of this revision.May 26 2022, 02:09

Harbormaster completed remote builds in B19156: Diff 33697.May 26 2022, 02:49

Please abandon, this is no longer relevant.

bytesofman requested changes to this revision.Sep 20 2022, 18:21

This revision now requires changes to proceed.Sep 20 2022, 18:21

emack abandoned this revision.Sep 21 2022, 00:25

Realized this is still relevant given airdrop function still relies on bch-api's getBlcokCount. Since this is referencing the old chronik client with long formats I'll submit a fresh diff.