Homec4science

Clean up Maniphest transaction rendering a bit more

Authored by epriestley <git@epriestley.com> on Sep 26 2013, 22:48.

Description

Clean up Maniphest transaction rendering a bit more

Summary:
Ref T2217. This partially retreads the ground from D7115.

  • We're rendering silly transactions about descriptions when creating tasks. Hide those.
  • Move the "created" transaction back to status. This fixes two things that are otherwise more of a mess than I'd anticipated:
  • Be explicit about action strength, so emails always say the most important thing in the subject.

Test Plan: Created and edited tasks, looked at resulting transactions, saw a cleaner transaction record.

Reviewers: btrahan

Reviewed By: btrahan

CC: aran

Maniphest Tasks: T2217

Differential Revision: https://secure.phabricator.com/D7141

Details

Committed
epriestley <git@epriestley.com>Sep 26 2013, 22:48
Pushed
aubortJan 31 2017, 17:16
Parents
rPH5677cd23bdec: Add storage and classes for CustomField in Differential
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHd13a3225634c: Clean up Maniphest transaction rendering a bit more (authored by epriestley <git@epriestley.com>).Sep 26 2013, 22:48