Skip to content

fix broken links in website docs - #3601

Merged
David Zimmermann (zimmy87) merged 5 commits into
masterfrom
clovett/brokenlinks
Apr 27, 2021
Merged

fix broken links in website docs#3601
David Zimmermann (zimmy87) merged 5 commits into
masterfrom
clovett/brokenlinks

Conversation

@lovettchris

Copy link
Copy Markdown
Contributor

Fixes: #
lots of broken links found with a webcrawler tool.

About

Fixing broken links.
This is extracted from the much bigger PR# 3546 so I can break that PR into easier to review parts.

How Has This Been Tested?

mkdocs build and the webcrawler tool.

Screenshots (if appropriate):

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good for the most part, but I think a few links are wrong and some confusing text elsewhere.

Comment thread docs/log_viewer.md Outdated
Comment thread docs/px4_setup.md Outdated
Comment thread docs/px4_sitl.md Outdated
Comment thread docs/px4_sitl.md Outdated
Comment thread docs/unreal_upgrade.md Outdated
@lovettchris Chris Lovett (lovettchris) changed the title fix broken links in website docs fix broken links in website docs [will delete this when the others are completed] Apr 20, 2021
@lovettchris Chris Lovett (lovettchris) changed the title fix broken links in website docs [will delete this when the others are completed] fix broken links in website docs Apr 20, 2021

@rajat2004 Rajat Singhal (rajat2004) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

distance_sensor.md should be added to the navigation in mkdocs.yml so that it appears in the sidebar. mkdocs build does give an info message about the missing page, but currently doesn't treat it as a warning

@lovettchris

Copy link
Copy Markdown
Contributor Author

Rajat Singhal (@rajat2004), fixed and I added a toc level for "Sensors" to group this content as I expect it will grow over time...
image

@zimmy87
David Zimmermann (zimmy87) merged commit 815fe8e into master Apr 27, 2021
@lovettchris
Chris Lovett (lovettchris) deleted the clovett/brokenlinks branch May 6, 2021 21:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants