Homec4science

Mark Maniphest notifications read if we send the user an email

Authored by epriestley <git@epriestley.com> on Oct 23 2012, 21:02.

Description

Mark Maniphest notifications read if we send the user an email

Summary:
See D3784, T1403. When we send a user an email and a notification from Maniphest, mark the notification as read.

(It would be nice to do the thing with multiplexMail() a little less hackily, but it gets very complicated to do correctly because we require handles but sometimes do not have an actor/user so I'm punting for now.)

Test Plan: Acted on a task, verified notification was marked read because I received an email.

Reviewers: vrana, btrahan

Reviewed By: vrana

CC: aran

Maniphest Tasks: T1403

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

Details

Committed
epriestley <git@epriestley.com>Oct 23 2012, 21:02
Pushed
aubortJan 31 2017, 17:16
Parents
rPH696a1b22babf: Make feed stories properly respect object policies
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH6b39af4022e2: Mark Maniphest notifications read if we send the user an email (authored by epriestley <git@epriestley.com>).Oct 23 2012, 21:02