Page MenuHomePhabricator

Don't prompt to mark notifications as read if we don't need to
ClosedPublic

Authored by epriestley on Aug 28 2017, 7:18 PM.
Tags
None
Referenced Files
F18922305: D18483.id.diff
Nov 9 2025, 3:18 PM
F18745662: D18483.diff
Oct 3 2025, 8:44 AM
F18566410: D18483.id.diff
Sep 9 2025, 2:36 PM
F18501457: D18483.diff
Sep 4 2025, 9:45 PM
F18404557: D18483.id44401.diff
Aug 29 2025, 10:39 PM
Unknown Object (File)
May 1 2025, 8:34 AM
Unknown Object (File)
May 1 2025, 8:34 AM
Unknown Object (File)
May 1 2025, 8:34 AM
Subscribers
None

Details

Summary

Fixes whatever task is tracking this junk, if one exists. Don't prompt unless there's a security issue.

Test Plan
  • Generated notifications from a test account.
  • Clicked "Mark All" from dropdown menu, no prompt.
  • Clicked "Mark All" from notifications screen, no prompt.
  • Command-Clicked "Mark All" from dropdown menu to open in new window, got normal prompt.

Diff Detail

Repository
rP Phabricator
Branch
noprompt
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 18188
Build 24455: Run Core Tests
Build 24454: arc lint + arc unit