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

Notifications page takes forever to load, or causes application error #2278

Closed
bugzilla-to-github opened this issue Mar 18, 2021 · 2 comments · Fixed by #3147
Closed

Notifications page takes forever to load, or causes application error #2278

bugzilla-to-github opened this issue Mar 18, 2021 · 2 comments · Fixed by #3147
Assignees
@mathjazz
Labels
notifications P2 We want to ship it soon, possibly in the current quarter performance task tech debt

Comments

@bugzilla-to-github
Copy link
Collaborator

This issue was created automatically by a script.

Bug 1699610

Bug Reporter: @flodolo
CC: @mathjazz

My notifications page takes forever to load, and in some cases triggers an application error. I guess I'm an edge case, having too many notifications (2703 right now).

The resulting HTML is 1.65 MB, and takes 17 seconds to load.

In particular, it looks like the page has no pagination for results, which might be the main cause.
@bugzilla-to-github
Copy link
Collaborator Author

Comment Author: @flodolo

I think this one is fixed?

@bugzilla-to-github
Copy link
Collaborator Author

Comment Author: @mathjazz

Nope, none of the speedups have landed.

(Any improvements you might have noticed are due to variable Heroku/AWS performance).

@mathjazz mathjazz changed the title [notifications] Notifications page takes forever to load, or causes application error Notifications page takes forever to load, or causes application error Nov 10, 2021
@mathjazz mathjazz removed their assignment Feb 28, 2023
@mathjazz mathjazz added task and removed bug labels Sep 25, 2023
@mathjazz mathjazz mentioned this issue Nov 22, 2023
Closed
@mathjazz mathjazz self-assigned this Mar 12, 2024
@mathjazz mathjazz mentioned this issue Mar 14, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mathjazz mathjazz

Labels
notifications P2 We want to ship it soon, possibly in the current quarter performance task tech debt
Projects
Pontoon 2024 Q1
Status: Performance Optimization (High Priority)
Pontoon Roadmap
Archived in project
Pontoon Committed Work (Deprecated)
Status: In Progress
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Speed up /notifications page mathjazz/pontoon
2 participants
@mathjazz @bugzilla-to-github