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

Refreshing notifications takes a long time #5027

Open
wakest opened this issue Apr 1, 2024 · 1 comment
Open

Refreshing notifications takes a long time #5027

wakest opened this issue Apr 1, 2024 · 1 comment
Labels
🧠 Backend Related to server logic and execution 🖐️ UX How something feels

Comments

@wakest
Copy link

wakest commented Apr 1, 2024

I click it and it looks like it works the first time, and then if I click it again it doesn't give any indication of working a second time

video of issue

tested on newest Firefox on macOS and pixelfed.social

@trwnh
Copy link
Member

trwnh commented Jun 3, 2024

It works for me, but it takes a long time (10-15 seconds) -- what might be happening for you is that the "refresh notifications" action takes so long that it is indistinguishable from the case where it works only once. Possibly the code is inefficient?

@trwnh trwnh added 🖐️ UX How something feels 🧠 Backend Related to server logic and execution labels Jun 3, 2024
@trwnh trwnh added this to the 0.14.0 - Polish milestone Jun 3, 2024
@trwnh trwnh changed the title refresh Notifications button only works the first time on page load Refreshing notifications takes a long time Jun 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🧠 Backend Related to server logic and execution 🖐️ UX How something feels
Projects
None yet
Development

No branches or pull requests

2 participants