[Cashtab] Add chaintip blockheight to wallet context
ClosedPublic
Actions

Authored by bytesofman on Jan 16 2024, 14:50.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABCd343d61ef250: [Cashtab] Add chaintip blockheight to wallet context

Summary

We need the chaintip blockheight to determine which coinbase utxos are not yet mature.

Add this info to wallet context.

If the info is unavailable, Cashtab will ignore all coinbase utxos. So, if there is some error in fetching it, log it to the console but do not lock the UI.

Test Plan

Add console.log(tipHeight`, tipHeight) between lines 241 and 242 in useWallet.js`
npm start, check that tipHeight is printed to the console.

Diff Detail

Repository

rABC Bitcoin ABC

Branch

add-chaintipheight-to-context

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 26398
Build 52366: Build Diff	cashtab-tests
Build 52365: arc lint + arc unit

Event Timeline

bytesofman created this revision.Jan 16 2024, 14:50

Herald added a reviewer: Restricted Project. · View Herald TranscriptJan 16 2024, 14:50

bytesofman published this revision for review.Jan 16 2024, 14:50

Harbormaster completed remote builds in B26398: Diff 44244.Jan 16 2024, 14:52

PiRK accepted this revision.Jan 16 2024, 14:57

This revision is now accepted and ready to land.Jan 16 2024, 14:57

Closed by commit rABCd343d61ef250: [Cashtab] Add chaintip blockheight to wallet context (authored by bytesofman). · Explain WhyJan 16 2024, 15:44

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABCd343d61ef250: [Cashtab] Add chaintip blockheight to wallet context.

Revision Contents
Changeset List

Path

Size

cashtab/

src/

hooks/

useWallet.js

13 lines

Diff 44244

View Options

cashtab/src/hooks/useWallet.js

[Cashtab] Add chaintip blockheight to wallet contextClosedPublicActions