Page MenuHomePhabricator
Differential D10495

[keyserver] Prevent self-mentions from pushing notifs to user
AbandonedPublic
Actions

Authored by rohan on Dec 29 2023, 10:02 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 8, 7:35 PM
Unknown Object (File)
Mon, Apr 8, 7:35 PM
Unknown Object (File)
Mon, Apr 8, 6:50 PM
Unknown Object (File)
Mar 15 2024, 9:52 AM
Unknown Object (File)
Mar 7 2024, 12:50 PM
Unknown Object (File)
Feb 14 2024, 5:07 AM
Unknown Object (File)
Feb 5 2024, 4:13 PM
Unknown Object (File)
Jan 30 2024, 7:41 PM
View All 12 Files
Subscribers
ashoat
tomek

Details

Reviewers
atul
ginsu
inka
Summary

To prevent users from getting a notif when they mention themselves, we should check if the message info creator is the same as the mentioned user ID.

Depends on D10494

Test Plan

Checked to make sure I didn't get a notif on the iOS device when I self-mentioned myself

Diff Detail

Repository
rCOMM Comm
Branch
ENG-6297
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

rohan created this revision.Dec 29 2023, 10:02 AM
Herald added subscribers: tomek, ashoat. · View Herald TranscriptDec 29 2023, 10:02 AM
rohan published this revision for review.EditedDec 29 2023, 10:08 AM
rohan abandoned this revision.

Actually, I don't think we need this. I did some testing without this change and I'm not getting notifications for any self-mentions (I was logged into 'ashoat' on native)

Harbormaster completed remote builds in B25406: Diff 35104.Dec 29 2023, 10:21 AM

Revision Contents
Changeset List

PathSize
keyserver/
src/
push/
3 lines

Diff 35104

View Options

keyserver/src/push/send.js

Loading...