-
Notifications
You must be signed in to change notification settings - Fork 126
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
Integrate screenshots in documentation #851
Comments
We've had a discussion somewhere (can't find where), where Daniel pointed out that the current setup might burn through our LFS quotas because people will download the image every time they consult our doc. @DJMcNab is this still a problem? |
We discussed it in office hours two weeks ago (https://docs.google.com/document/d/1cL4CywyDJ-AdTyQTYTySYJWCsaq0nT50uV9rliFM6zU/edit?tab=t.0#heading=h.evjimzdnrevx). It might still be an issue - the docs from GitHub are a bit unclear. The decision we made was to launch it as currently planned, then:
Certainly if someone has the will to move to a properly hosted solution on linebender.org (the exact technical details there are TBD, because we want to keep the screenshots for all previous tags available), then that would be welcome. |
Could we publish the screenshots in a Github release and link to those? I don't think Github release downloads are metered? |
#832 add a way to add screenshots to documentation
What we need after that PR is merged is to:
The text was updated successfully, but these errors were encountered: