Skip to content
Snippets Groups Projects

Optimize Notifications Rendering

Merged Shpuld Shpludson requested to merge feat/optimize-amount-of-notifs-to-render into develop
All threads resolved!

It works like this:

  • keeps count of how many seen notifications it should display per notifications column (we can have multiple thanks to interactions)
  • increases the number when loading older to show more
  • puts the number back to default whenever clicking "Read!", since we can be sure you're on top whenever you click it. This cuts off bloat in DOM from all the older notifications (which are still in store)
  • when clicking load older while the older are still in store, it just increases the number to display, only fetching more when the number is greater than what's in store already.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • added 1 commit

    Compare with previous version

  • Shpuld Shpludson added 6 commits

    added 6 commits

    Compare with previous version

  • Shpuld Shpludson resolved all threads

    resolved all threads

  • mentioned in commit ac90d4c5

  • Please register or sign in to reply
    Loading