Page MenuHomePhabricator

Fix an outdated HTML anchor link in Diffusion table of contents
ClosedPublic

Authored by epriestley on Sep 15 2017, 12:34 PM.
Tags
None
Referenced Files
F14081106: D18609.diff
Fri, Nov 22, 6:26 PM
F14079433: D18609.diff
Fri, Nov 22, 8:09 AM
Unknown Object (File)
Thu, Nov 21, 1:01 PM
Unknown Object (File)
Mon, Nov 18, 8:18 AM
Unknown Object (File)
Wed, Nov 13, 6:04 PM
Unknown Object (File)
Sat, Nov 9, 9:49 PM
Unknown Object (File)
Wed, Nov 6, 2:44 AM
Unknown Object (File)
Oct 20 2024, 10:28 AM
Subscribers
None

Details

Summary

See https://discourse.phabricator-community.org/t/navigating-to-changed-files-in-diffusion-does-not-work-anymore/433.

In D18465, I updated these but this hard-coded the anchor for some reason (???) and I missed it while grep-ing.

Test Plan

Viewed a commit (/rXYZaaaa) and clicked a file link in the table of contents. Got modern #change-... anchor and navigation into the document.

Diff Detail

Repository
rP Phabricator
Branch
anchor1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 18459
Build 24856: Run Core Tests
Build 24855: arc lint + arc unit