Page MenuHomePhabricator

Always link to "#5" (not "#comment-5") for "T123#5"
ClosedPublic

Authored by epriestley on Mar 14 2014, 9:39 PM.
Tags
None
Referenced Files
F19040222: D8542.id20267.diff
Wed, Nov 26, 11:52 AM
F18834779: D8542.id.diff
Oct 26 2025, 10:45 AM
F18808798: D8542.id20269.diff
Oct 19 2025, 8:43 AM
F18744930: D8542.id20267.diff
Oct 3 2025, 4:41 AM
F18734692: D8542.id.diff
Sep 30 2025, 10:59 PM
F18107107: D8542.diff
Aug 10 2025, 11:40 PM
F18082338: D8542.id20269.diff
Aug 5 2025, 5:21 AM
F17988692: D8542.id20267.diff
Aug 1 2025, 11:10 PM
Subscribers

Details

Summary

Fixes T3976. Long ago, some applications used "#comment-5" instead of "#5" for transaction/comment anchors. Now everything (I think?) uses "#5"; this is the style used by ApplicationTransactions.

This might break some very old, explcit T123#comment-5 links, or off-site links to the comment-N anchors, but all that stuff generally got renumbered when we migrated anyway and getting you to the right object is like 95% of the job.

Test Plan

Verified that T123#5 now links to #5.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Always link to "#5" (not "#comment-5") for "T123#5".
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 14 2014, 9:43 PM
epriestley updated this revision to Diff 20269.

Closed by commit rP3bea0958dd31 (authored by @epriestley).