Description

When a commit URL could mean several different commits, Phabricator shows all the commits that fit the ambiguous identifier.

https://secure.phabricator.com/rPbd3c23

There are two commits this matches, rPbd3c2355e8 and rPbd3c239d5a.

Mostly fixed in T13001. Ambiguous commit URLs now show this:

Screen Shot 2017-12-05 at 11.27.46 AM.png (567×1 px, 72 KB)

TODO: The repository in the crumb is currently not a link, but should link to the repository that the commit belongs to. This likely involves adding another mode to the buildCrumbList method in DiffusionController.

Related Objects

Mentioned In: T13001: Ambiguous commits should prompt you to select from options
Mentioned Here: rPbd3c2355e8e2: Match usernames in any case in commit message object lists (CC, Reviewers, etc.
rPbd3c239d5aad: Remove a stray `setActor()` on EdgeEditor
T13001: Ambiguous commits should prompt you to select from options

Event Timeline

lpriestley created this task.Dec 5 2017, 7:39 PM

epriestley mentioned this in T13001: Ambiguous commits should prompt you to select from options.Dec 22 2017, 2:58 PM

joshuaspence added a subscriber: joshuaspence.Dec 26 2017, 7:48 AM

	F5311176: Screen Shot 2017-12-05 at 11.27.46 AM.png
	Dec 5 2017, 7:39 PM

Crumbs on a page that shows options for an ambiguous commit identifier should link to the repositoryOpen, Needs TriagePublicActions

Description

Related Objects

Event Timeline

Crumbs on a page that shows options for an ambiguous commit identifier should link to the repository
Open, Needs TriagePublic
Actions