Page MenuHomePhabricator

Allow projects to be "watched", sort of a super-subscribe
ClosedPublic

Authored by epriestley on May 18 2014, 9:54 PM.
Tags
None
Referenced Files
F13239929: D9185.id21828.diff
Wed, May 22, 8:07 AM
F13237691: D9185.id.diff
Tue, May 21, 2:22 PM
F13237526: D9185.diff
Tue, May 21, 1:51 PM
F13220660: D9185.diff
Sun, May 19, 1:51 AM
F13219439: D9185.id21816.diff
Sat, May 18, 6:23 PM
F13202671: D9185.diff
Tue, May 14, 10:59 PM
F13185659: D9185.diff
Sat, May 11, 3:07 AM
Unknown Object (File)
Tue, May 7, 5:24 AM
Subscribers

Details

Summary

Ref T4967. Adds a "Watch" relationship to projects, which is stronger than member/subscribed.

Specifically, when a task is tagged with a project, we'll include all project watchers in the email/notifications. Normally we don't include projects unless they're explicitly CC'd, or have some other active role in the object (like being a reviewer or auditor).

This allows you to closely follow a project without needing to write a Herald rule for every project you care about.

Test Plan
  • Watched/unwatched a project.
  • Tested the watch/subscribe/member relationships:
    • Watching implies subscribe.
    • Joining implies subscribe.
    • Leaving implies unsubscribe + unwatch.
    • You can't unsubscribe until you unwatch (slightly better would be unsubscribe implies unwatch, but this is a bit tricky).
  • Watched a project, then recevied email about a tagged task without otherwise being involved.

Event Timeline

epriestley retitled this revision from to Allow projects to be "watched", sort of a super-subscribe.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
  • Before using members of the watch list, verify they can see the object.
btrahan edited edge metadata.
This revision is now accepted and ready to land.May 19 2014, 6:59 PM
epriestley updated this revision to Diff 21859.

Closed by commit rP3a3155426898 (authored by @epriestley).