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.

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

Add notification toasts/sonner #15

Open
zaknesler opened this issue May 18, 2024 · 0 comments
Open

Add notification toasts/sonner #15

zaknesler opened this issue May 18, 2024 · 0 comments

Comments

@zaknesler
Copy link
Owner

Should be in the style of Sonner.

The most important here is "refreshing feeds (X of Y)" which automatically increments. Ideally this'll work with the following webhook notifications (received in order):

  • RefreshingFeeds: feed_uuids
  • Each feed: RefreshedFeed: feed_uuid

Can create a simple state machine to count when we've refreshed all the feeds, and then can update the same toast popup with the number of feeds as they get created. Then, once all feeds are refreshed, we can invalidate the local queries. It might also be worth it to update the feeds on a timeout while we are refreshing so that we

  1. Update the list as the feeds are refreshing, in case we have many feeds to refresh and it takes a few seconds
  2. Don't invalidate the stats/feeds/entries more than we have to (doing it for each feed would be way too many API calls).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant