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
F18843599: D8403.diff
Tue, Oct 28, 11:38 PM
F18741879: D8403.id.diff
Thu, Oct 2, 4:26 PM
F18741200: D8403.id19970.diff
Thu, Oct 2, 9:13 AM
F18735983: D8403.diff
Oct 1 2025, 4:20 AM
F18703320: D8403.id19957.diff
Sep 28 2025, 3:37 AM
F18702545: D8403.diff
Sep 28 2025, 12:34 AM
F18595841: D8403.id19957.diff
Sep 12 2025, 11:02 PM
F18594967: D8403.diff
Sep 12 2025, 7:58 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