Page MenuHomePhabricator
Differential D8403

Add an explcit "Changes Planned" state for Differential
ClosedPublic
Actions

Authored by epriestley on Mar 5 2014, 2:18 AM.
Tags
None
Referenced Files
F19706094: D8403.diff
Wed, Feb 11, 9:08 AM
F19545986: D8403.diff
Sat, Jan 24, 9:25 PM
F19522794: D8403.id.diff
Sat, Jan 17, 10:49 AM
F19520059: D8403.id.diff
Fri, Jan 16, 10:48 AM
F19517423: D8403.id.diff
Thu, Jan 15, 11:06 AM
F19509387: D8403.id19970.diff
Jan 10 2026, 7:05 PM
F18865752: D8403.id19957.diff
Nov 3 2025, 1:47 PM
F18843599: D8403.diff
Oct 28 2025, 11:38 PM
View All Files
Subscribers
aran

Details

Reviewers
btrahan
Maniphest Tasks
T2222: Implement ApplicationTransactions in Differential
T4481: Differential Update Feedback / Rough Edges / Status
Commits
Restricted Diffusion Commit
rP4ef87eeac832: Add an explcit "Changes Planned" state for Differential
Summary

Ref T2222. Ref T4481. This fixes the issue where "Plan Changes" could immediately trigger a state change (e.g., back to accepted) because of state-based transitions out of the NEEDS_REVISION state.

Test Plan

Planned changes an "accepted" revision, it didn't immediately return to being accepted.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

btrahan accepted this revision.Mar 5 2014, 6:36 PM
btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 5 2014, 6:36 PM
epriestley closed this revision.Mar 5 2014, 6:48 PM

Closed by commit rP4ef87eeac832 (authored by @epriestley).

Revision Contents
Changeset List

Diff 19970

View Options

src/applications/differential/constants/DifferentialRevisionStatus.php

Loading...
View Options

src/applications/differential/controller/DifferentialRevisionViewController.php

Loading...
View Options

src/applications/differential/editor/DifferentialRevisionEditor.php

Loading...
View Options

src/applications/differential/editor/DifferentialTransactionEditor.php

Loading...
View Options

src/applications/differential/view/DifferentialRevisionListView.php

Loading...
Phabricator · Built by Phacility