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

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

Tags

None

Referenced Files

	F15467491: D8707.diff
	Thu, Apr 3, 3:32 PM

	F15449329: D8707.id20653.diff
	Fri, Mar 28, 9:33 AM

	F15449318: D8707.id20652.diff
	Fri, Mar 28, 9:29 AM

	F15429534: D8707.id.diff
	Mon, Mar 24, 2:52 AM

	F15414533: D8707.diff
	Thu, Mar 20, 1:02 AM

	F15388018: D8707.id20652.diff
	Sat, Mar 15, 3:12 AM

	F15384405: D8707.diff
	Fri, Mar 14, 7:47 PM

	Unknown Object (File)
	Feb 24 2025, 11:38 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