Homec4science

Maniphest + Differential - don't show transaction preview until user interacts…

Authored by Bob Trahan <btrahan@phacility.com> on May 20 2015, 22:54.

Description

Maniphest + Differential - don't show transaction preview until user interacts with the form.

Summary: Fixes T4846. These are one off (for now) since they have various crazy actions with them. I think this will get unified and more cleaned up when we refine the UI for taking multiple actions at once, etc.

Test Plan: noted no "commented on x" in either maniphest or differential. starting making a comment and noted prevew showed. started adding a subscriber (added to tokenizer) and preview showed.

Reviewers: epriestley

Reviewed By: epriestley

Subscribers: Korvin, epriestley

Maniphest Tasks: T4846

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

Details

Committed
Bob Trahan <btrahan@phacility.com>May 20 2015, 22:54
Pushed
aubortJan 31 2017, 17:16
Parents
rPHbb0004fd41bc: Make default start value of new events to be the next whole hour and the end…
Branches
Unknown
Tags
Unknown

Event Timeline

Bob Trahan <btrahan@phacility.com> committed rPH9de39c12a2b4: Maniphest + Differential - don't show transaction preview until user interacts… (authored by Bob Trahan <btrahan@phacility.com>).May 20 2015, 22:54