Don't link "Dxxx" on Differential revision pages
ClosedPublic
Actions

Authored by epriestley on Feb 1 2017, 2:53 AM.

Tags

None

Referenced Files

	F15522169: D17283.id41577.diff
	Sun, Apr 20, 6:26 PM

	F15480398: D17283.diff
	Tue, Apr 8, 1:57 PM

	F15480393: D17283.diff
	Tue, Apr 8, 1:56 PM

	F15467241: D17283.id41578.diff
	Thu, Apr 3, 1:50 PM

	F15454590: D17283.id41578.diff
	Sat, Mar 29, 7:12 PM

	F15451967: D17283.id41577.diff
	Sat, Mar 29, 12:55 AM

	F15444809: D17283.id.diff
	Thu, Mar 27, 11:24 AM

	F15439384: D17283.diff
	Wed, Mar 26, 7:18 AM

View All Files

Subscribers

None

Details

Reviewers

chad

Maniphest Tasks

T12027: Replace "Next Step" Differential advice with comprehensive instructions on "Land Revision"

Commits

rP9b92e56dfc03: Don't link "Dxxx" on Differential revision pages

Summary

Ref T12027. See T12043 for discussion.

Test Plan

Double-clicked "Dxxx" to select it.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 1 2017, 2:53 AM

chad accepted this revision.Feb 1 2017, 2:53 AM

This revision is now accepted and ready to land.Feb 1 2017, 2:53 AM

Harbormaster completed remote builds in B15475: Diff 41577.Feb 1 2017, 2:54 AM

epriestley mentioned this in T12027: Replace "Next Step" Differential advice with comprehensive instructions on "Land Revision".Feb 1 2017, 2:54 AM

Closed by commit rP9b92e56dfc03: Don't link "Dxxx" on Differential revision pages (authored by epriestley, committed by epriestley). · Explain WhyFeb 1 2017, 2:55 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

controller/

DifferentialRevisionViewController.php

2 lines

Diff 41578

View Options

src/applications/differential/controller/DifferentialRevisionViewController.php

Don't link "Dxxx" on Differential revision pagesClosedPublicActions