Skip to content

Update faq.md to use MD rather than raw HTML - #746

Open
DespicableGoose wants to merge 2 commits into
mavlink:masterfrom
DespicableGoose:fix-docs-faq
Open

Update faq.md to use MD rather than raw HTML#746
DespicableGoose wants to merge 2 commits into
mavlink:masterfrom
DespicableGoose:fix-docs-faq

Conversation

@DespicableGoose

Copy link
Copy Markdown
Contributor

Currently, visiting the generated page (https://mavlink.io/en/about/faq.html) has broken links.
This is because the document is explicitly written in raw html, so VitePress does no meaningful transformation from md to generated HTML pages.
Since the literal links to other documentation pages are explicitly encoded in the HTML and they link to .md files, these links break upon actual page deployment/generation, since the md files get translated to HTML, and thus the .md files cannot be found, leading to 404 errors.
This PR should fix this issue. Sorry for the verbosity of my explanation...

Currently, visiting the generated page (https://mavlink.io/en/about/faq.html) has broken links.
This is because the document is explicitly written in raw html, so VitePress does no meaningful transformation from md to generated HTML pages.
Since the literal links to other documentation pages are explicitly encoded in the HTML and they link to .md files, these links break upon actual page deployment/generation, since the md files get translated to HTML, and thus the .md files cannot be found, leading to 404 errors.
This commit/PR should fix this issue. Sorry for the verbosity of my explanation...
@DespicableGoose

Copy link
Copy Markdown
Contributor Author

This is the only page I could find in the documentation with this issue when investigating. I initially noticed the issue naturally, when browsing documentation for other purposes and finding that the links on the FAQ page were invalid. However, when trying to find more issues of this nature specifically, not just broken links in general, I couldn't find any. I wasn't particularly thorough when searching though, I will admit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant