HomePhabricator

PhabricatorSlug
1175784d5d84Unpublished

Unpublished Commit · Learn More

No further details are available.

Description

PhabricatorSlug

Summary:
This is to be used in Phame so the logic is shared where possible. The change has three main things going on

I think maybe we should punt on unicode here for quite a bit -- http://www.456bereastreet.com/archive/201006/be_careful_with_non-ascii_characters_in_urls/ -- but we'll be well-positioned to add it with the code here.

Test Plan: used phriction to create, edit, view documents. used a tool (codemod) for the codebase sweeping

Reviewers: epriestley

Reviewed By: epriestley

CC: aran

Differential Revision: https://secure.phabricator.com/D2195

Details

Provenance
btrahanAuthored on
Reviewer
epriestley
Differential Revision
Restricted Differential Revision
Parents
rP01907bcccc31: Allow "Test Plan" to be disabled in config
Branches
Unknown
Tags
Unknown

Event Timeline