Closed sudojunior closed 2 years ago
Reason for the change
page.path
page.url
*.md
Description - direct from #1303
The reason for the complex replacement join is due to how liquid identifies and handle strings (and also the order of execution)
split, slice, join(/), append(/), append(related[1])
split, slice, append(related[1]), join(/)
apijavascriptnext
api/javascript/next/
Code examples No core documentation was changed.
Checklist
References
Language navigator faults / Page mismatch also ended up being tracked on this spreadsheet despite saying they actively weren't.
Side notes
span.disabled
Thank you for noticing the bug and making a PR for the fix. Merging.
Reason for the change
page.path
was attempting to replacepage.url
with the*.md
extension that doesn't exist.Description - direct from #1303
The reason for the complex replacement join is due to how liquid identifies and handle strings (and also the order of execution)
split, slice, join(/), append(/), append(related[1])
split, slice, append(related[1]), join(/)
Placing append before a join breaks the sequence and turns the array of strings into a concatenated string before the join can run resulting inapijavascriptnext
instead ofapi/javascript/next/
Code examples No core documentation was changed.
Checklist
References
Side notes
span.disabled
for language navigator no longer exists, and I cannot determine when it was removed or if it existed to begin with.