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 links in core/cartopy/cartopy #144

Closed
jukent opened this issue Sep 17, 2021 · 3 comments · Fixed by #145
Closed

Broken links in core/cartopy/cartopy #144

jukent opened this issue Sep 17, 2021 · 3 comments · Fixed by #145
Labels
content Content related issue

Comments

@jukent
Copy link
Contributor

jukent commented Sep 17, 2021

Link-checker is failing PR #131 due to broken links in core/cartopy - which was not changed in this PR. So we have some broken links upstream?

(    core/cartopy: line   16) broken    https://scitools.org.uk/cartopy/docs/latest/crs/projections.html#cartopy-projections - 404 Client Error: Not Found for url: https://scitools.org.uk/cartopy/docs/latest/crs/projections.html
(core/cartopy/cartopy: line 650005) broken    https://scitools.org.uk/cartopy/docs/latest/crs/projections.html - 404 Client Error: Not Found for url: https://scitools.org.uk/cartopy/docs/latest/crs/projections.html
@jukent
Copy link
Contributor Author

jukent commented Sep 17, 2021

This seems like an easy fix if anyone knows where the link is supposed to point.
My guess is that SciTools recently updated their site structure and that's why this link is broken. Similarly some SciTools google results are broken links now too.

In which case a quick fix would be to set the links to an older version instead of "latest"
i.e. https://scitools.org.uk/cartopy/docs/v0.15/crs/projections.html and https://scitools.org.uk/cartopy/docs/v0.15/crs/projections.html#cartopy-projections

@jukent jukent added the content Content related issue label Sep 17, 2021
@jukent jukent mentioned this issue Sep 17, 2021
@dopplershift
Copy link
Contributor

The right fix I think is to update to the new link for v0.20: https://scitools.org.uk/cartopy/docs/latest/reference/crs.html#list-of-projections

@jukent
Copy link
Contributor Author

jukent commented Sep 20, 2021

The right fix I think is to update to the new link for v0.20: https://scitools.org.uk/cartopy/docs/latest/reference/crs.html#list-of-projections

Okay great! I couldn't find the new path that would point to a comparable page.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content Content related issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants