Page MenuHomePhabricator

Let Phabricator parse either "Dxxx" (no URI) or "https://secure.phabricator.com/Dxxx" (full URI) in Differential Revisions
Closed, ResolvedPublic

Description

Currently, I believe we require "Differential Revision" to strictly have recognized, full URIs. Instead, it should accept full URIs or no URI (it's possible it does actually do this, but we should verify that's really the case if so), and only reject full URIs for unrecognized domains.

This is a blocker for eventually making the full URIs optional.

In modern Phabricator, we have tons of unqualified references (like "Fixes Txxx") so qualifying a tiny fraction of the references isn't really useful any more.

Revisions and Commits

Restricted Differential Revision
Restricted Differential Revision

Event Timeline

epriestley claimed this task.
epriestley raised the priority of this task from to Normal.
epriestley updated the task description. (Show Details)
epriestley added projects: Arcanist, Differential.
epriestley added subscribers: epriestley, cburroughs.
epriestley added a revision: Restricted Differential Revision.May 7 2015, 12:59 PM
epriestley added a revision: Restricted Differential Revision.
epriestley closed this task as Resolved by committing Restricted Diffusion Commit.May 7 2015, 6:10 PM
epriestley added a commit: Restricted Diffusion Commit.
epriestley added a commit: Restricted Diffusion Commit.