Page MenuHomePhabricator

Deduplicate application-level notifications from Aphlict
ClosedPublic

Authored by epriestley on Apr 17 2017, 9:46 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Mar 27, 11:58 AM
Unknown Object (File)
Tue, Mar 5, 2:09 AM
Unknown Object (File)
Dec 27 2023, 10:41 AM
Unknown Object (File)
Dec 27 2023, 10:41 AM
Unknown Object (File)
Dec 27 2023, 10:41 AM
Unknown Object (File)
Dec 21 2023, 5:42 PM
Unknown Object (File)
Dec 21 2023, 5:42 PM
Unknown Object (File)
Dec 21 2023, 5:42 PM
Subscribers
None

Details

Summary

Fixes T12564. We already had some code which seems to deal with this properly, it just wasn't getting used.

Assign each application-level notification a unique ID, then ignore messages with duplicate IDs.

Test Plan
  • In browser A, loaded /T123.
  • In browser B, loaded /T123.
  • Made a comment as B.
  • Saw notification as A.
  • Mashed "Replay" a bunch.
  • Before patch: piles of duplicate notifications.
  • After patch: no duplicates.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Apr 17 2017, 9:54 PM
This revision was automatically updated to reflect the committed changes.