Differential D8719

Hide "Land to hosted git" button for now
ClosedPublic
Actions

Authored by epriestley on Apr 8 2014, 12:41 PM.

Tags

None

Referenced Files

	F15395432: D8719.diff
	Sun, Mar 16, 6:59 AM

	F15389745: D8719.id20678.diff
	Sat, Mar 15, 5:36 AM

	F15375280: D8719.id20674.diff
	Wed, Mar 12, 10:22 PM

	F15375279: D8719.diff
	Wed, Mar 12, 10:22 PM

	F15333651: D8719.diff
	Sat, Mar 8, 3:46 AM

	Unknown Object (File)
	Thu, Feb 27, 8:28 PM

	Unknown Object (File)
	Feb 24 2025, 11:39 AM

	Unknown Object (File)
	Feb 24 2025, 11:39 AM

Subscribers

Details

Reviewers

btrahan
chad

Maniphest Tasks

T182: Commit into repository directly from differential

Commits

Restricted Diffusion Commit
rPe8e62f82ceae: Hide "Land to hosted git" button for now

Summary

Ref T182. This feature rarely/never works and is on the balance enormously confusing to users (see https://github.com/facebook/phabricator/issues/566). If installs have somehow made it sort of work, they can comment this line out for now until we have time to make this work more reasonably.

Test Plan

Looked at a revision in Differential.

Diff Detail

Repository

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley updated this revision to Diff 20674.Apr 8 2014, 12:41 PM

epriestley retitled this revision from to Hide "Land to hosted git" button for now.

epriestley updated this object.

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

epriestley added a reviewer: btrahan.

epriestley added a task: T182: Commit into repository directly from differential.

Herald added a subscriber: epriestley.Apr 8 2014, 12:41 PM

chad accepted this revision.Apr 8 2014, 2:56 PM

chad added a reviewer: chad.

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

Closed by commit rPe8e62f82ceae (authored by @epriestley).

btrahan added inline comments.Apr 8 2014, 5:20 PM

src/applications/differential/landing/DifferentialLandingToHostedGit.php
128	*eventually

epriestley mentioned this in D18150: Remove old "Landing Strategy" code.Jun 23 2017, 3:10 PM

epriestley mentioned this in rP219ae8b6c950: Remove old "Landing Strategy" code.Jun 23 2017, 3:14 PM

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

landing/

DifferentialLandingToHostedGit.php

5 lines

Diff 20678

src/applications/differential/landing/DifferentialLandingToHostedGit.php

Loading...