HomePhabricator

Deduplicate application-level notifications from Aphlict

Description

Deduplicate application-level notifications from Aphlict

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.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T12564

Differential Revision: https://secure.phabricator.com/D17710

Details

Provenance
epriestleyAuthored on Apr 17 2017, 9:42 PM
epriestleyPushed on Apr 17 2017, 10:55 PM
Reviewer
chad
Differential Revision
D17710: Deduplicate application-level notifications from Aphlict
Parents
rP953ab039acc0: Disable auto-zoom on mobile form UIs
Branches
Unknown
Tags
Unknown
Tasks
T12564: Handle duplicate realtime messages from Aphlict gracefully
Build Status
Buildable 16536
Build 22032: Run Core Tests