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

Remove all around notifications #4515

Closed
evavirseda opened this issue Dec 17, 2024 · 0 comments
Closed

Remove all around notifications #4515

evavirseda opened this issue Dec 17, 2024 · 0 comments
Labels
tooling Issues related to the Tooling team wallet-dashboard Issues related to the Wallet Dashboard dApp

Comments

@evavirseda
Copy link
Contributor

evavirseda commented Dec 17, 2024 •
edited
Loading

Now that we're using toasts, there is still some debris around notifications, and some places still use them instead of toasts. We should update these files and remove all debris related to notifications, such as hooks, store, types, etc.
Context
@evavirseda evavirseda changed the title Remove notification hooks and function debris Remove all around notifications logic Dec 17, 2024
@evavirseda evavirseda changed the title Remove all around notifications logic Remove all around notifications Dec 17, 2024
@evavirseda evavirseda added tooling Issues related to the Tooling team wallet-dashboard Issues related to the Wallet Dashboard dApp labels Dec 17, 2024
This was referenced Dec 17, 2024
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
tooling Issues related to the Tooling team wallet-dashboard Issues related to the Wallet Dashboard dApp
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@evavirseda