Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Check links in the user guide #1089

Closed
chalin opened this issue Jul 4, 2022 · 2 comments · Fixed by #1172
Closed

Check links in the user guide #1089

chalin opened this issue Jul 4, 2022 · 2 comments · Fixed by #1172
Assignees
@chalin
Labels
CI/infra CI & infrastructure e0-minutes Effort < 60 min p1-high

Comments

@chalin
Copy link
Collaborator

chalin commented Jul 4, 2022

No description provided.

@chalin chalin added e0-minutes Effort < 60 min CI/infra CI & infrastructure p1-high labels Jul 4, 2022
@deining
Copy link
Collaborator

deining commented Jul 6, 2022 •
edited
Loading

Please note, that this issue which was raised in the example site already, they suggest hyperlink as link checking tool.
Another approach is to use the netflify-checklinks-plugin(https://github.com/Munter/netlify-plugin-checklinks) for link checking. I experimented with this plugin in this branch a while ago. Results were promising, but the branch ist not fully ready for merging yet.

@chalin
Copy link
Collaborator Author

chalin commented Jul 7, 2022

Thanks for the background info @deining. I'll look into that as soon as I can.

@chalin chalin self-assigned this Aug 15, 2022
@chalin chalin mentioned this issue Aug 15, 2022
Merged
@chalin chalin mentioned this issue Aug 15, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@chalin chalin

Labels
CI/infra CI & infrastructure e0-minutes Effort < 60 min p1-high
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Check links in the user guide chalin/docsy
2 participants
@chalin @deining