Page MenuHomePhabricator
Differential D17575

[Cashtab] No double notifications for SLP partial lists
ClosedPublic
Actions

Authored by bytesofman on Wed, Jan 22, 23:21.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Commits
rABC63dc3deda632: [Cashtab] No double notifications for SLP partial lists
Summary

This was missed in the first pass of removing double notifications. Does not apply to ALP as ALP does not require the "ad prep tx" -- only SLP. NFT listing already has this handled.

Test Plan

npm test

Diff Detail

Repository
rABC Bitcoin ABC
Branch
more-double-notifications-gone
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 32183
Build 63856: Build Diffcashtab-tests
Build 63855: arc lint + arc unit

Event Timeline

bytesofman created this revision.Wed, Jan 22, 23:21
Herald added a reviewer: Restricted Project. · View Herald TranscriptWed, Jan 22, 23:21
Harbormaster completed remote builds in B32183: Diff 52338.Wed, Jan 22, 23:26
bytesofman published this revision for review.Wed, Jan 22, 23:29
bytesofman added inline comments.
cashtab/src/components/Etokens/Token/index.tsx
1846

We still have this info in the successful listing notification below

Fabien accepted this revision.Thu, Jan 23, 08:16
This revision is now accepted and ready to land.Thu, Jan 23, 08:16
Closed by commit rABC63dc3deda632: [Cashtab] No double notifications for SLP partial lists (authored by bytesofman). · Explain WhyThu, Jan 23, 08:32
This revision was automatically updated to reflect the committed changes.
bytesofman added a commit: rABC63dc3deda632: [Cashtab] No double notifications for SLP partial lists.

Revision Contents
Changeset List

PathSize
cashtab/
src/
components/
Etokens/
Token/
6 lines

Diff 52338

View Options

cashtab/src/components/Etokens/Token/index.tsx

Loading...