Put revisions waiting on other reviewers in their own bucket
ClosedPublic
Actions

Authored by epriestley on Feb 27 2017, 6:17 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, May 4, 8:11 PM

	Unknown Object (File)
	Fri, May 3, 9:03 AM

	Unknown Object (File)
	Fri, May 3, 1:38 AM

	Unknown Object (File)
	Sat, Apr 27, 3:05 PM

	Unknown Object (File)
	Thu, Apr 25, 2:43 AM

	Unknown Object (File)
	Fri, Apr 19, 6:17 PM

	Unknown Object (File)
	Fri, Apr 19, 3:15 PM

	Unknown Object (File)
	Thu, Apr 11, 9:57 AM

View All Files

Subscribers

None

Details

Reviewers

chad

Maniphest Tasks

T12323: Revisions show in Other Revisions when acceptance status is unclear

Commits

rP6c21646b5f33: Put revisions waiting on other reviewers in their own bucket

Summary

Fixes T12323. See that task for discussion.

Test Plan

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 27 2017, 6:17 PM

chad accepted this revision.Feb 27 2017, 6:18 PM

This revision is now accepted and ready to land.Feb 27 2017, 6:18 PM

Harbormaster completed remote builds in B15801: Diff 41900.Feb 27 2017, 6:18 PM

Closed by commit rP6c21646b5f33: Put revisions waiting on other reviewers in their own bucket (authored by epriestley, committed by epriestley). · Explain WhyFeb 27 2017, 6:47 PM

This revision was automatically updated to reflect the committed changes.

jcarrillo7 mentioned this in T12235: Differential wrongly says I "Must Review" reviews rejected by group.Mar 1 2017, 12:27 AM

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

query/

DifferentialRevisionRequiredActionResultBucket.php

26 lines

Diff 41903

View Options

src/applications/differential/query/DifferentialRevisionRequiredActionResultBucket.php

Put revisions waiting on other reviewers in their own bucketClosedPublicActions