HomePhorge

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, 12:12 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rPb8b701faf734: Clarify language when Autoclose is disabled for a repository
Branches
Unknown
Tags
Unknown

Event Timeline