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.

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

Broken link check #212

Open
Pete-Robelou opened this issue Jun 22, 2021 · 2 comments · Fixed by #226
Open

Broken link check #212

Pete-Robelou opened this issue Jun 22, 2021 · 2 comments · Fixed by #226
Assignees

Comments

@Pete-Robelou
Copy link
Collaborator

No description provided.

@adamduncan adamduncan self-assigned this Jun 22, 2021
@adamduncan adamduncan removed their assignment Jun 22, 2021
@adamduncan
Copy link
Collaborator

adamduncan commented Jun 22, 2021

I've run an automated broken link checker across the site.

First run I attemped to run across the whole site, but after indexing across the /news/blog section ran into the 10's of thousand without end in site, I decided we'll have to exclude it to get any meaningful results across the new site content.

Have captured results here in spreadsheet for review. We could do without another spreadsheet link 😄 Have included on a new page, along with our other spreadsheet: https://docs.google.com/spreadsheets/d/1HM6g0YDHxaNsBam9wzDPeCrmaW7zR1MKEY2xBuc_1II/edit#gid=121972092

@Thingee would suggest if you're able to correct any to their rightful place, then we can be confident the majority of the non-Blog site is all linked correctly. Perhaps for the blog import content (which one might expect there are some legacy broken links throughout), we'll have to correct as we go

Thingee added a commit to Thingee/ceph.io that referenced this issue Jun 22, 2021
@adamduncan adamduncan linked a pull request Jun 23, 2021 that will close this issue
@adamduncan
Copy link
Collaborator

Ticked a few off and merged, thanks @Thingee.

Moving back to "To do" to keep a reference to the remaining links to be corrected.

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 a pull request may close this issue.

3 participants