[Cashtab] Re-implement 11008 after UI landing
ClosedPublic
Actions

Authored by bytesofman on Feb 16 2022, 14:23.

Details

Reviewers

emack
johnkuney

Group Reviewers

Restricted Project

Commits

rABC0d4252ea6bf1: [Cashtab] Re-implement 11008 after UI landing

Summary

See D10970 and D11008

Changes from D11008 were reverted inadvertently in D10970. This diff brings them back in.

Test Plan

npm test and ensure all pass
Review D10970 to confirm no other non-design-related changes

Diff Detail

Repository

rABC Bitcoin ABC

Branch

re-implement-11008

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 18288
Build 36381: Build Diff	cashtab-tests
Build 36380: arc lint + arc unit

Event Timeline

bytesofman created this revision.Feb 16 2022, 14:23

Herald added a reviewer: Restricted Project. · View Herald TranscriptFeb 16 2022, 14:23

bytesofman requested review of this revision.Feb 16 2022, 14:23

Harbormaster completed remote builds in B18288: Diff 32407.Feb 16 2022, 14:26

johnkuney accepted this revision.Feb 16 2022, 17:57

This revision is now accepted and ready to land.Feb 16 2022, 17:57

Closed by commit rABC0d4252ea6bf1: [Cashtab] Re-implement 11008 after UI landing (authored by Joey King <joseph.roy.king@gmail.com>). · Explain WhyFeb 16 2022, 18:07

This revision was automatically updated to reflect the committed changes.

Joey King <joseph.roy.king@gmail.com> added a commit: rABC0d4252ea6bf1: [Cashtab] Re-implement 11008 after UI landing.

Revision Contents
Changeset List

Path

Size

web/

cashtab/

src/

hooks/

__tests__/

useBCH.test.js

134 lines

Diff 32407

View Options

web/cashtab/src/hooks/tests/useBCH.test.js

[Cashtab] Re-implement 11008 after UI landingClosedPublicActions