Page MenuHomePhabricator

Improve "unblock task" feed stories
ClosedPublic

Authored by epriestley on Aug 12 2014, 4:22 PM.
Tags
None
Referenced Files
F18777445: D10235.diff
Sat, Oct 11, 3:01 AM
F18712081: D10235.id.diff
Mon, Sep 29, 5:16 AM
F18637241: D10235.id24628.diff
Sep 17 2025, 6:42 AM
F18637240: D10235.id24628.diff
Sep 17 2025, 6:41 AM
F18625840: D10235.id24627.diff
Sep 15 2025, 11:43 PM
F18619124: D10235.diff
Sep 15 2025, 1:47 AM
F18608834: D10235.diff
Sep 14 2025, 2:07 AM
F18503979: D10235.id.diff
Sep 4 2025, 11:31 PM
Subscribers

Details

Summary

Fixes T5184. Fixes T5008. Three issues with stories/notifications about changing the status of tasks which block other tasks:

Bad Feed Stories

  • Problem: Feed story rendering was confusing (T5184).
  • Solution: fix it to provide context.

Too Many Feed Stories

  • Problem: Feed gets a story for the original task's close ("a closed x"), and a story for each blocked task ("a closed x, a task blocking y").
  • "Solution": Punt. These are redundant in the full feed but not in filtered feeds. Right solution is display-time aggregation. No users have really complained about this.

Too Many Notifications

  • Problem: Users subscribed to both tasks get notified about the clsoe, and also about the unblocked task. These notifications are redundant.
  • "Solution": Punt. This is easy to fix by silencing notifications for the sub-editor, but I'm worried it would be confusing. Users haven't complained. Display-time aggregation might be a better fix.
Test Plan

Screen_Shot_2014-08-12_at_9.22.20_AM.png (95×612 px, 15 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped