Page MenuHomePhabricator

When a task card is edited, emit update events for old boards and parent boards
ClosedPublic

Authored by epriestley on Jul 24 2019, 5:29 PM.
Tags
None
Referenced Files
F15468145: D20680.id49333.diff
Thu, Apr 3, 8:40 PM
F15456016: D20680.diff
Sun, Mar 30, 6:26 AM
F15453971: D20680.id49323.diff
Sat, Mar 29, 3:25 PM
F15446148: D20680.diff
Thu, Mar 27, 5:24 PM
F15413151: D20680.diff
Wed, Mar 19, 5:15 PM
F15367700: D20680.diff
Tue, Mar 11, 8:37 PM
Unknown Object (File)
Mar 1 2025, 12:17 PM
Unknown Object (File)
Feb 11 2025, 8:20 AM
Subscribers
None

Details

Summary

Ref T4900. When a card is edited, we currently emit an update notification for all the projects the task is tagged with. This isn't quite the right set:

  • We want to emit notifications for projects the task was previously tagged with, so it can be removed from boards it should no longer be part of.
  • We want to emit notifications for ancestors of projects the task is or was tagged with, so parent project boards can be updated.
  • However, we don't need to emit notifications for projects that don't actually have workboards.

Adjust the notification set to align better to these rules.

Test Plan
  • Removal of Parent Project: Edited a task on board "A > B", removing the "B" project tag. Saw board A update in another window.
  • Normal Update: Edited a task title on board X, saw board X update in another window.
  • Used bin/aphlict debug to inspect the notification set, saw generally sensible-seeming data going over the wire.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable