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 Instructor Notes #1501

Closed
marklcrowe opened this issue Mar 3, 2023 · 7 comments
Closed

Broken links in Instructor Notes #1501

marklcrowe opened this issue Mar 3, 2023 · 7 comments

Comments

@marklcrowe
Copy link

The Building Skills with Practice episode (https://preview.carpentries.org/instructor-training/instructor/02-practice-learning.html) includes an instructor note with examples of mental models. The links in this note are broken, and give a 403 Forbidden error.

@brownsarahm
Copy link
Contributor

I filed a sandpaper issue because this is an issue in how links are converted in the build process and though we could manually fix them, a long term more maintainable solution is for this to be fixes in the build tools.

A manual workaround is to delete the solo instructor from the path so that:

https://preview.carpentries.org/instructor-training/instructor/fig/array-math.png

becomes

https://preview.carpentries.org/instructor-training/fig/array-math.png

We could manually update all of the paths to be the full path of the rendered images for now, but that fix would break again when we move from preview. back to this being the main version.

@zkamvar
Copy link
Contributor

zkamvar commented Mar 11, 2023

I will make sure to have the next version of {sandpaper} with this fix up next week. I have been a bit underwater coming back to a breaking dependency and a mountain of emails.

@zkamvar
Copy link
Contributor

zkamvar commented Mar 11, 2023

Another solution is to render them as actual images and add extra width attributes to make them smaller:

![Array Math](fig/array-math.png){width="10%"}

@brownsarahm
Copy link
Contributor

I like the idea of the rendering them small, but then would still want to allow clicking on them to see the full version (though some people may know that you can right click and open images in new tabs, not all will).

@zkamvar
Copy link
Contributor

zkamvar commented Mar 14, 2023

I've pushed a fix in sandpaper 0.11.9 and it should be available widely in about an hour. Once it is updated on https://carpentries.r-universe.dev/sandpaper, you can go to https://github.com/carpentries/instructor-training/actions/workflows/sandpaper-main.yaml to rebuild the lesson to fix the links.

@zkamvar
Copy link
Contributor

zkamvar commented Mar 14, 2023

And I apologise for this taking longer than expected. The last few weeks have thrown a few challenges my way.

@karenword
Copy link
Contributor

This is now fixed. Thanks @marklcrowe !

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

No branches or pull requests

4 participants