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
Unknown Object (File)
Sat, Jan 18, 10:48 AM
Unknown Object (File)
Sat, Jan 18, 12:34 AM
Unknown Object (File)
Fri, Jan 17, 8:14 PM
Unknown Object (File)
Sun, Jan 12, 3:59 PM
Unknown Object (File)
Sat, Jan 11, 5:31 PM
Unknown Object (File)
Fri, Jan 10, 7:00 PM
Unknown Object (File)
Sun, Jan 5, 7:57 AM
Unknown Object (File)
Sat, Jan 4, 9:17 PM
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