-
-
Notifications
You must be signed in to change notification settings - Fork 655
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
Migrate to Unsplash API #1085
Comments
Also have no idea how we're gonna migrate the current image references. |
Tim mentions the old ID's are still relevant: https://x.com/TimVanDijck/status/1795455355981304124 |
Hi @driesvints Wanted to check if you're working on this issue because I have a working solution
Solution Options
My Queries
Also please provide feedback or if you think this should be done differently |
If the urls remain constant then yeah we can indeed set them in a table column. @joedixon already worked a bit on a command to go over each article here: #1090
It's done here: laravel.io/app/Models/Article.php Line 109 in 8e873eb
Or did you mean how it was saved? There's no code for that. I manually copied the identifier and added it to the record in the database. We want to eventually develop an image picker but that's quite a bit of work. |
Yes I meant how it was saved. Thank you for the clarification. |
Unsplash decided to sunset their https://source.unsplash.com domain without any warning so now all image fetching is broken. We'll have to migrate to their API but think that'll get us into API limits. I don't know if the generated urls from Unsplash are long lived. If they are we could maybe save them in the database otherwise we might have to run a scheduled job to refresh them during their TTL.
The text was updated successfully, but these errors were encountered: