Don't show 'needs review' on DifferentialObjects
ClosedPublic
Actions

Authored by chad on Apr 4 2014, 8:55 PM.

Tags

None

Referenced Files

	F13992336: D8707.id20653.diff
	Tue, Oct 22, 3:52 PM

	F13972372: D8707.id20653.diff
	Oct 17 2024, 6:38 PM

	F13955038: D8707.id20653.diff
	Oct 13 2024, 11:55 PM

	Unknown Object (File)
	Oct 13 2024, 5:51 PM

	Unknown Object (File)
	Oct 10 2024, 7:01 AM

	Unknown Object (File)
	Oct 1 2024, 9:04 AM

	Unknown Object (File)
	Sep 21 2024, 8:39 AM

	Unknown Object (File)
	Sep 7 2024, 3:11 AM

View All Files

Subscribers

epriestley

Korvin

Details

Reviewers

epriestley

Commits

Restricted Diffusion Commit
rP3b33089de7f6: Don't show 'needs review' on DifferentialObjects

Summary

This text is overly repetitive and is not super important. Keeps the other states. Also

Easier to parse reviewers now
Mobile is less janky

Test Plan

reload my list of diffs

Diff Detail

Repository

rP Phabricator

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

chad updated this revision to Diff 20652.Apr 4 2014, 8:55 PM

chad retitled this revision from to Don't show 'needs review' on DifferentialObjects.

chad updated this object.

chad edited the test plan for this revision. (Show Details)

chad added a reviewer: epriestley.

Herald added subscribers: Korvin, epriestley.Apr 4 2014, 8:55 PM

chad edited the test plan for this revision. (Show Details)Apr 4 2014, 8:55 PM

epriestley accepted this revision.Apr 4 2014, 8:59 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Apr 4 2014, 8:59 PM

Closed by commit rP3b33089de7f6 (authored by @chad).

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

view/

DifferentialRevisionListView.php

5 lines

Diff 20653

View Options

src/applications/differential/view/DifferentialRevisionListView.php

Don't show 'needs review' on DifferentialObjectsClosedPublicActions