Make updates of rejected revisions behave correctly again
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F19069291: D8402.id.diff
	Nov 30 2025, 1:34 PM

	F19061578: D8402.diff
	Nov 29 2025, 12:58 PM

	F18850020: D8402.id.diff
	Oct 30 2025, 1:52 PM

	F18845558: D8402.diff
	Oct 29 2025, 12:58 PM

	F18822551: D8402.id19969.diff
	Oct 23 2025, 7:24 AM

	F18757318: D8402.id19956.diff
	Oct 5 2025, 4:51 PM

	F18717436: D8402.id.diff
	Sep 29 2025, 1:58 PM

	F18705698: D8402.diff
	Sep 28 2025, 12:41 PM

View All Files

Subscribers

aran

Details

Reviewers

btrahan

Maniphest Tasks

T4481: Differential Update Feedback / Rough Edges / Status
T2222: Implement ApplicationTransactions in Differential

Commits

Restricted Diffusion Commit
rP9da6ec208111: Make updates of rejected revisions behave correctly again

Summary

Ref T2222. Ref T4481. Specifically:

When a revision is updated, change all "Reject" reviewers to "Reject Prior".
Change status to "Needs Review".
Update the state logic to account for this properly.

Test Plan

Created a revision as user A, with B as a reviewer.
Rejected as B.
Updated the revision as A.
Saw revision in "needs review" state, with B as a "Rejected Prior" reviewer.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

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

btrahan edited edge metadata.

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

Closed by commit rP9da6ec208111 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

constants/

DifferentialReviewerStatus.php

9 lines

editor/

DifferentialTransactionEditor.php

45 lines

view/

DifferentialReviewersView.php

12 lines

Diff 19969

View Options

src/applications/differential/constants/DifferentialReviewerStatus.php

View Options

src/applications/differential/editor/DifferentialTransactionEditor.php

View Options

src/applications/differential/view/DifferentialReviewersView.php

Make updates of rejected revisions behave correctly againClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 19969

src/applications/differential/constants/DifferentialReviewerStatus.php

src/applications/differential/editor/DifferentialTransactionEditor.php

src/applications/differential/view/DifferentialReviewersView.php

Make updates of rejected revisions behave correctly again
ClosedPublic
Actions

Revision Contents
Changeset List