-
-
Notifications
You must be signed in to change notification settings - Fork 16
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
Exercism Stories Image Script #151
Comments
@iHiD We also need to decide what format to store the thumbnails in. They're currentl huge (2550x1440 or something like that) |
@iHiD Do we want to store all images as jpegs or as pngs? I see that the existing files are all jpegs, but Jonathan's file is a png. |
Currently, I take the image, put it through squoosh and get JPGs (MozJPEG to be precise) at about 85% quality. PNGs are too big files for photos. (See settings below). I then upload to s3 and put the URL in the config. I'd suggest that a script looks checks PRs for files in a certain directory and if it finds them, puts them through squoosh, uploads to s3 (we probably want to set the filename to the slug of the dir) and deletes the original file in the PR. Then sets the image_url to that predictable path with the correct filename. |
@ErikSchierboom could I create a folder and add the images to that folder for the script to process? |
This issue outlines a process for uploading, storing and accessing the assets required for the distribution of Exercism Stories to major streaming platforms.
The Current Process
The process of getting all of the different aspects (listed below) together for a podcast typically takes half a day (once the recording is complete). I've sought to clarify the process in detail. Any feedback or ways to optimise are welcome.
exercism/blog
repo with the following information and create a PR that is reviewed and that populates the Exercism Stories page -> see example hereWhat is Needed
image_url
- currently it is not implemented@iHiD what do you think the process should be in light of all the various priorities.
@ErikSchierboom is this an adequate description of what is required?
The text was updated successfully, but these errors were encountered: