Restore "Land Revision" action to UI
ClosedPublic
Actions

Authored by epriestley on Jun 26 2017, 1:46 PM.

Tags

None

Referenced Files

	F19824619: D18154.diff
	Mar 8 2026, 1:27 AM

	F19824614: D18154.diff
	Mar 8 2026, 1:24 AM

	F19713332: D18154.id43680.diff
	Feb 11 2026, 10:18 PM

	F19692124: D18154.id43680.diff
	Feb 10 2026, 5:09 AM

	F19659424: D18154.id43680.diff
	Feb 7 2026, 1:10 AM

	F19218415: D18154.diff
	Dec 17 2025, 9:05 AM

	F19068349: D18154.diff
	Nov 30 2025, 11:54 AM

	F18866664: D18154.diff
	Nov 3 2025, 6:32 PM

View All Files

Subscribers

None

Details

Reviewers

chad

Commits

rP7cb67dc7257e: (stable) Restore "Land Revision" action to UI
rPd4632f4b78a0: Restore "Land Revision" action to UI

Summary

This was accidentally caught in the crossfire in D18150. This is stable enough to formalize instead of adding with an event hook.

Test Plan

Looked at a candidate revision, saw "Land Revision" appear in UI again.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Jun 26 2017, 1:46 PM

chad accepted this revision.Jun 26 2017, 1:47 PM

This revision is now accepted and ready to land.Jun 26 2017, 1:47 PM

Harbormaster completed remote builds in B17569: Diff 43679.Jun 26 2017, 1:48 PM

Closed by commit rPd4632f4b78a0: Restore "Land Revision" action to UI (authored by epriestley, committed by epriestley). · Explain WhyJun 26 2017, 1:50 PM

This revision was automatically updated to reflect the committed changes.

epriestley mentioned this in rP219ae8b6c950: Remove old "Landing Strategy" code.Jun 26 2017, 1:51 PM

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

controller/

DifferentialRevisionViewController.php

23 lines

Diff 43680

View Options

src/applications/differential/controller/DifferentialRevisionViewController.php

Restore "Land Revision" action to UIClosedPublicActions