Page MenuHomePhabricator
Differential D11540

[Cashtab] [Chronik Websocket Notifications] [Part 9] Display eToken notifications from websocket msg
ClosedPublic
Actions

Authored by bytesofman on May 30 2022, 23:14.

Details

Reviewers
emack
Group Reviewers
Restricted Project
Commits
rABC81be83fa7a0a: [Cashtab] [Chronik Websocket Notifications] [Part 9] Display eToken…
Summary

T2447

Depends on D11539

Display eToken received notifications when seen by websocket

Test Plan

npm start
Send an eToken tx to your wallet
Observe two notifications: 1 quick from the websocket, and another some seconds after from the old way

Repeat with an eToken tx new to this wallet

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bytesofman created this revision.May 30 2022, 23:14
Herald added a reviewer: Restricted Project. · View Herald TranscriptMay 30 2022, 23:14
bytesofman requested review of this revision.May 30 2022, 23:14
bytesofman updated this revision to Diff 33736.

Removing line used in debugging

Harbormaster completed remote builds in B19190: Diff 33735.May 30 2022, 23:17
Harbormaster completed remote builds in B19191: Diff 33736.
bytesofman edited the summary of this revision. (Show Details)May 31 2022, 17:53
bytesofman added a parent revision: D11539: [Cashtab] [Chronik Websocket Notifications] [Part 8] Check cache for eToken info.
bytesofman added a child revision: D11541: [Cashtab] [Chronik Websocket Notifications] [Part 10] If ws connection active, hide api notifications.
emack added a subscriber: emack.Jun 7 2022, 00:29

as per telegram

image.png (405×927 px, 57 KB)

bytesofman updated this revision to Diff 33846.Jun 7 2022, 01:17

Rebase

Harbormaster completed remote builds in B19276: Diff 33846.Jun 7 2022, 01:21
emack accepted this revision.Jun 7 2022, 05:43
This revision is now accepted and ready to land.Jun 7 2022, 05:43
Closed by commit rABC81be83fa7a0a: [Cashtab] [Chronik Websocket Notifications] [Part 9] Display eToken… (authored by bytesofman). · Explain WhyJun 7 2022, 21:06
This revision was automatically updated to reflect the committed changes.
bytesofman added a commit: rABC81be83fa7a0a: [Cashtab] [Chronik Websocket Notifications] [Part 9] Display eToken….

Revision Contents
Changeset List

PathSize
web/
cashtab/
src/
hooks/
33 lines

Diff 33877

View Options

web/cashtab/src/hooks/useWallet.js

Loading...