-
Notifications
You must be signed in to change notification settings - Fork 1
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
Placeholder images for places that do not have an image #19
Comments
There's already a placeholder for places without pictures, those with this exact picture have it as their related image in the DB I think, can @JasperDR confirm ? |
That is correct, this can be changed in the DB by uploading another picture and putting that one in the DB. I'm already working on a function to return the images from backend. |
Okay, I don’t know the technical details, what needs to happen is that we
As soon as possible because there is a rights issue. |
As long as the hot linked image is deleted from the DB, the placeholder will be applied :) |
cleared 'photos' table from DB |
Great! However, who thought it was a good idea to put a 1mb picture as a placeholder? This should be a couple of mb’s max. Not considering the placeholder debacle fixed just yet ;) https://leuvenleestapp.be/assets/img/placeholder_locations.jpg |
We need to add a placeholder image for places that do not have a photo. Right now, it is hot linked. http://www.leuven.be/binaries/Stadhuis-CoprightLaylaAerts_tcm16-24398.jpg
This has major high priority, assigning @huhu1030 and @elroyK because we cannot just hotlink a copyrighted image. Please fix ASAP :)
The text was updated successfully, but these errors were encountered: