Page MenuHomePhabricator

Support `M123/456` mock references in Remarkup
ClosedPublic

Authored by epriestley on Jun 15 2014, 7:11 PM.
Tags
None
Referenced Files
F18087266: D9555.id22907.diff
Wed, Aug 6, 3:16 AM
F18086953: D9555.id22916.diff
Wed, Aug 6, 1:22 AM
F18065129: D9555.id22904.diff
Mon, Aug 4, 2:01 PM
F18057131: D9555.diff
Mon, Aug 4, 8:54 AM
F17948364: D9555.id22916.diff
Thu, Jul 31, 8:51 PM
F17948054: D9555.id22907.diff
Thu, Jul 31, 7:56 PM
F17940239: D9555.diff
Thu, Jul 31, 4:08 AM
Unknown Object (File)
Jul 3 2025, 7:03 AM
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).