Page MenuHomePhabricator

Support `M123/456` mock references in Remarkup
ClosedPublic

Authored by epriestley on Jun 15 2014, 7:11 PM.
Tags
None
Referenced Files
F15403154: D9555.id22916.diff
Tue, Mar 18, 2:02 AM
F15401268: D9555.id22907.diff
Mon, Mar 17, 4:39 PM
F15398830: D9555.id22916.diff
Mon, Mar 17, 1:49 AM
F15397506: D9555.id.diff
Sun, Mar 16, 8:40 PM
F15396113: D9555.id22904.diff
Sun, Mar 16, 11:57 AM
F15395372: D9555.diff
Sun, Mar 16, 6:20 AM
F15390981: D9555.diff
Sat, Mar 15, 7:39 AM
F15352779: D9555.id22907.diff
Mon, Mar 10, 6:35 PM
Subscribers

Details

Summary

Fixes T4729. This form is a little fluff, but we show it in the URI when you click an anchor on the page, and doing so seems desirable. I think it's reasonable to support this form, given that it appears in the URI.

Test Plan

Wrote some stuff like M60, M60/71, M60/72/, M60/73/#13 and saw it all get picked up and rendered/linked properly.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Support `M123/456` mock references in Remarkup.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Jun 15 2014, 7:53 PM
epriestley updated this revision to Diff 22916.

Closed by commit rP99e354952241 (authored by @epriestley).