[Cashtab] Patch syntax error in useEffect
ClosedPublic
Actions

Authored by bytesofman on Feb 29 2024, 13:06.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABCa4a4fb45d18b: [Cashtab] Patch syntax error in useEffect

Summary

The dependent param should be in an array.

Test Plan

npm test

Diff Detail

Repository

rABC Bitcoin ABC

Branch

patch-useeffect

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 27534
Build 54631: Build Diff	cashtab-tests
Build 54630: arc lint + arc unit

Event Timeline

bytesofman created this revision.Feb 29 2024, 13:06

Herald added a reviewer: Restricted Project. · View Herald TranscriptFeb 29 2024, 13:06

bytesofman mentioned this in D15566: [Cashtab] Move cashtabCache to cashtabState.Feb 29 2024, 13:06

Harbormaster completed remote builds in B27534: Diff 45755.Feb 29 2024, 13:18

bytesofman published this revision for review.Feb 29 2024, 13:19

PiRK accepted this revision.Feb 29 2024, 14:42

This revision is now accepted and ready to land.Feb 29 2024, 14:42

Closed by commit rABCa4a4fb45d18b: [Cashtab] Patch syntax error in useEffect (authored by bytesofman). · Explain WhyFeb 29 2024, 15:24

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABCa4a4fb45d18b: [Cashtab] Patch syntax error in useEffect.

Revision Contents
Changeset List

Path

Size

cashtab/

src/

hooks/

useWallet.js

2 lines

Diff 45755

View Options

cashtab/src/hooks/useWallet.js

[Cashtab] Patch syntax error in useEffectClosedPublicActions