HomePhabricator

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

Description

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

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.

Reviewers: chad

Reviewed By: chad

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

Details

Provenance
epriestleyAuthored on Aug 28 2017, 7:12 PM
epriestleyPushed on Aug 28 2017, 8:05 PM
Reviewer
chad
Differential Revision
D18483: Don't prompt to mark notifications as read if we don't need to
Parents
rPb8b701faf734: Clarify language when Autoclose is disabled for a repository
Branches
Unknown
Tags
Unknown
Build Status
Buildable 18189
Build 24456: Run Core Tests