Page MenuHomePhabricator

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

Authored by btrahan on May 19 2015, 8:29 PM.
Tags
None
Referenced Files
F14018228: D12936.diff
Tue, Nov 5, 6:37 AM
F14011484: D12936.diff
Fri, Nov 1, 2:13 AM
Unknown Object (File)
Sep 17 2024, 8:17 AM
Unknown Object (File)
Sep 15 2024, 12:09 PM
Unknown Object (File)
Sep 10 2024, 4:04 PM
Unknown Object (File)
Sep 9 2024, 8:18 PM
Unknown Object (File)
Sep 5 2024, 4:37 AM
Unknown Object (File)
Sep 4 2024, 8:11 PM
Subscribers

Details

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.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

btrahan retitled this revision from to Maniphest + Differential - don't show transaction preview until user interacts with the form..
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.May 19 2015, 11:06 PM
This revision was automatically updated to reflect the committed changes.