-
-
Notifications
You must be signed in to change notification settings - Fork 289
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
HTTP vs https links #1237
Comments
@karenword this should be added as one of the global fixes in the update |
I realise now I could have indeed applied a similar fix to all files in the repo. Would you like me to submit a PR for this? Or is it something better left to someone with better knowledge of possible future major re-designs? |
Thank you @Bisaloo ! We are currently working on updates in a different repo, so I have created an issue there. We had done a pass-through on these at one point, but clearly a number of links were missed! For most links I think it will be easiest for us to to address changes in the updates. However, if you happen across any others that cause problems urgently, please do submit those. Thanks again -- this was a very useful catch! |
hi @karenword |
Added HTTPS and direct links to avoid issues with redirections and HTTP inspired by carpentries/instructor-training#1237
Originally raised through partial fix by @Bisaloo ! in #1236 (comment)
The text was updated successfully, but these errors were encountered: