Paths

Table of Contentst

Add "Changes Planned" and "In Preparation" states for revisions
ClosedPublic
Actions

Authored by epriestley on Mar 5 2014, 1:12 AM.

Tags

None

Referenced Files

	F13283529: D8400.id19954.diff
	Sun, Jun 2, 3:44 PM

	F13282283: D8400.diff
	Sun, Jun 2, 11:53 AM

	F13226987: D8400.id19968.diff
	Mon, May 20, 12:34 AM

	F13215958: D8400.id.diff
	Fri, May 17, 9:21 PM

	F13212258: D8400.diff
	Fri, May 17, 6:32 AM

	F13210064: D8400.id19954.diff
	Fri, May 17, 3:43 AM

	F13201748: D8400.diff
	Tue, May 14, 3:38 PM

	F13194638: D8400.diff
	Sun, May 12, 9:44 PM

View All Files

Subscribers

aran

• weijingjaylin

Details

Reviewers

btrahan

Maniphest Tasks

T2543: Add a formal "Draft" / "Not Yet Ready for Review" state to Differential
T2222: Implement ApplicationTransactions in Differential

Commits

rARC7b6b24154b71: Add "Changes Planned" and "In Preparation" states for revisions

Summary

Ref T2222. Discussion in T4481. Ref T2543.

Primarily, this will let me fix some of the rough edges that came out of ApplicationTransactions and state-based transitions.

I'm also adding a constant for T2543 while I'm in here.

Test Plan

Used grep to look for any weird special behavior, didn't see any.

Diff Detail

Repository

rARC Arcanist

Branch

arcnew

Lint

Lint Passed

Unit

No Test Coverage

Event Timeline

btrahan accepted this revision.Mar 5 2014, 6:31 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Mar 5 2014, 6:31 PM

Closed by commit rARC7b6b24154b71 (authored by @epriestley).

what does "in preparation" mean? how to move it to "needs review" status?

Revision Contents
Changeset List

Path

Size

src/

differential/

constants/

ArcanistDifferentialRevisionStatus.php

16 lines

Diff 19954

View Options

src/differential/constants/ArcanistDifferentialRevisionStatus.php

Add "Changes Planned" and "In Preparation" states for revisionsClosedPublicActions