Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

In app tx related notifications improvements #16338

Open
Tracked by #14636
saledjenic opened this issue Sep 16, 2024 · 0 comments · May be fixed by #16732
Open
Tracked by #14636

In app tx related notifications improvements #16338

saledjenic opened this issue Sep 16, 2024 · 0 comments · May be fixed by #16732
Assignees
@saledjenic
Labels
feature wallet-team
Milestone
2.32.0 Beta

Comments

@saledjenic
Copy link
Contributor

saledjenic commented Sep 16, 2024 •
edited
Loading

Improve notifications within the app when the tx is placed/pending/confirmed.

At this moment notifications are a bit intertwined and many of them are missing in the app that we should have by the Figma file:

Signals that need to be revised:

const SIGNAL_OWNER_TOKEN_SENT* = "ownerTokenSent"
const SIGNAL_TRANSACTION_SENDING_COMPLETE* = "transactionSendingComplete"
const SIGNAL_TRANSACTION_STATUS_CHANGED* = "transactionStatusChanged"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@saledjenic saledjenic

Labels
feature wallet-team
Projects
Status Desktop/Mobile Board
Status: In Progress
Milestone
2.32.0 Beta
Development

Successfully merging a pull request may close this issue.

fix(wallet): in app tx related notifications improvements status-im/status-desktop
2 participants
@alaibe @saledjenic