xF2 Add-on Alert Improvements by Xon 2.12.2

• Rework alert pruning to better scale for large & busy forums

Click to expand...

• Fix that alerts css would be injected after extra.less, which made styling difficult

Click to expand...

• Fix that mark read/unread would remove the checkbox on the alerts page
• Push fauxBlockLink to be just around the per-template render to not overlap the minor info row

• Adjust locking ordering in markUserAlertsRead to hopefully be less deadlock prone
• Add sanity checking to unread alerts page to automatically rebuild alert counts on no unread alerts being seen

• Make global "mark read" work without a overlay or redirect to a new page, and reload alerts on the existing page when invoked
• Handle alerts unviewed/unread being out of sync so trigger summarization
• For reaction summary alerts
  • Ensure reactions have stable sorting (new summarizes only)
  • On generating summary alerts, ensure they are treated as just-read correctly in the UI.
  • On viewing details, Fix the new alert star was being put under the avatar incorrectly

• Fix access check when unsummarizing alerts. This could allow deleting arbitrary alerts for other users, but did not leak contents of that alert.
• Fix unsummarizating alerts where not correctly marking the alert as unviewed
• Remove redundant separator when floating mark read/unread right
• Update phrase used by the reaction summary for a single user giving multiple reactions

• When pruning alerts; Reduce maximum batch size, and back-off by 1 second between prune attempts

Click to expand...

• Fix unexpected extra queries when a user doesn't have unread alerts

Even if "Alerts pop-up skips mark read" is set, mark alerts with the content type "user" as read when the alert pop-up is opened. As these alerts never get marked as read by visiting content

• Patch hourly alert cleanup to better respect max job run time

Click to expand...