Page MenuHomePhabricator

[Cashtab] [BCH deprecation] Pt 1/2 - Implement local HD node derivePath logic
AbandonedPublic

Authored by emack on Oct 31 2022, 03:27.

Details

Reviewers
bytesofman
Group Reviewers
Restricted Project
Summary

T2370

As a prerequisite to deprecating BCH.HDNode.derivePath, this diff implements this locally within Cashtab.

Test Plan
  • npm test
  • npm start
  • create a new wallet and observe 3 x 'BCH-JS node output matches localNode' console log messages relfecting the active wallet's Path245, Path145 and Path1899 derivations

Diff Detail

Repository
rABC Bitcoin ABC
Branch
hdnode
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 20768
Build 41199: Build Diffcashtab-tests
Build 41198: arc lint + arc unit

Event Timeline

emack requested review of this revision.Oct 31 2022, 03:27

It's not clear what we are accomplishing here since masterHDNode is still being derived from BCH before calling the deriveAccount function. Fix it at the top level and then carrying it down.

This revision now requires changes to proceed.Oct 31 2022, 16:45