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
F14069875: D9185.diff
Wed, Nov 20, 10:25 AM
F14067720: D9185.diff
Tue, Nov 19, 4:22 PM
F14055325: D9185.diff
Sat, Nov 16, 10:25 AM
F14043522: D9185.diff
Tue, Nov 12, 10:52 AM
F14042584: D9185.diff
Tue, Nov 12, 4:33 AM
F14027765: D9185.diff
Fri, Nov 8, 9:07 AM
F13994587: D9185.diff
Wed, Oct 23, 7:13 AM
F13973403: D9185.id21859.diff
Oct 18 2024, 12:47 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).