Open llewelld opened 12 months ago
I think I might have missed the image part (I guess the proposed changes include having little images for the carousel), but regardless of that: I had assumed we'd just need to store the UUIDs (which could have been in env variables) and then use a fixed-in-template placeholders otherwise.
Having said that, I have no strong opinion in either direction, but just wanted to offer an alternative solution that could avoid creating a model if we prefer to not have one 🙂
Summary
Currently the uuids and image filenames for the story carousel on the home page are read from the carousel.json file.
Although it makes sense (I think) to store the placeholder stories in a file (because they should be available for use even if the database is cleared) it would make much more if the uuids for the real stories were stored in the databse.
This has two benefits:
What needs to be done?
get_carousel_stories()
helper method to use the database entries rather than reading them from thestories
entry in carousel.json.Who can help?