Page MenuHomePhabricator
Create Task
Maniphest T8052

Calendar feed story should include description of event
Closed, WontfixPublic
Actions

Description

I filled out a description, but it's not in the feed story.

pasted_file (181×321 px, 30 KB)

Event Timeline

chad created this task.May 4 2015, 12:14 AM
chad raised the priority of this task from to Needs Triage.
chad updated the task description. (Show Details)
chad added a project: Calendar.
chad added subscribers: chad, lpriestley, epriestley.
lpriestley moved this task from Backlog to Future Work (v2+) on the Calendar board.May 6 2015, 4:18 PM
epriestley added a comment.May 6 2015, 4:20 PM

Specifically:

  • Other, more useful applications (Maniphest, Differential) don't currently do this.
  • PhabricatorApplicationTransactionFeedStory doesn't really support doing this the right way.
    • Audit implements it OK.
    • Pholio and Ponder implement it but sort of show the wrong data (the description at view-time, not at create-time, which is misleading).
epriestley triaged this task as Wishlist priority.May 7 2015, 1:48 PM
lpriestley moved this task from Future Work (v2+) to Runners Up on the Calendar board.May 25 2015, 6:33 PM
lpriestley moved this task from Runners Up to Future Work (v2+) on the Calendar board.Jun 16 2015, 3:27 AM
epriestley closed this task as Wontfix.Nov 3 2016, 6:54 PM
epriestley claimed this task.

I'm just going to WONTFIX this for now since Maniphest and Differential don't show descriptions either and it's not clear to me that event descriptions are significantly more valuable than task/revision/etc descriptions. Open to changing this, but we should either change it everywhere or have some consistent rule for when they do/don't show.

I think the description cache on Pholio/Ponder is also obvious enough to rise to the level of being a bug, but I'm not sure if we should fix that or just remove the descriptions. I don't feel like I miss the descriptions in Differential/Maniphest today, at least, and Pholio seems similar-ish.

Phabricator · Built by Phacility