mirror of
https://github.com/discourse/discourse.git
synced 2024-11-26 19:00:32 -06:00
b7b61d4b56
We previously used ConsolidateNotifications with a threshold of 1 to re-use an existing notification and bump it to the top instead of creating a new one. It produces some jumpiness in the user notification list, and it relies on updating the `created_at` attribute, which is a bit hacky. As a better alternative, we're introducing a new plan that deletes all the previous versions of the notification, then creates a new one. |
||
|---|---|---|
| .. | ||
| consolidate_notifications.rb | ||
| consolidation_plan.rb | ||
| consolidation_planner.rb | ||
| delete_previous_notifications.rb | ||