MuckRock / muckrock

MuckRock's source code - Please report bugs, issues and feature requests to info@muckrock.com
https://www.muckrock.com
GNU Affero General Public License v3.0
114 stars 22 forks source link

Project images don't work #467

Closed morisy closed 9 years ago

morisy commented 9 years ago

Tried two different images, they both show up in the preview but not in the actual page: https://www.muckrock.com/project/troubling-foia-practices-2/ https://www.muckrock.com/project/lets-get-some-berghers-1/

allanlasser commented 9 years ago

Looks like a bad URL is all. The files exist in /project_images/ S3 directory but the page is looking for /media/project_images/.

mitchelljkotler commented 9 years ago

Did you use the static tag to get the path?

allanlasser commented 9 years ago

I used the Media tag. When I ran collectstatic locally, the project_images directory was created in the my local /static/media directory. But looking in the S3, there is no media directory and the *_images folders are at the bucket root. Do you know why this is?

As an aisde, I'm not using the thumbnail plugin ATM.

mitchelljkotler commented 9 years ago

Hmm... so "static" are files you check in to git, collectstatic then takes them and moves them to S3 (or where ever they are served from locally). "meida" files are files uploaded through django, and should automatically be put in S3 (or where ever they go locally). So its possible we have something configured incorrectly. But I'll take a look when I get a chance to see if I can tell whats going on.

On Wed, Jul 8, 2015 at 12:00 PM, Allan Lasser notifications@github.com wrote:

I used the Media tag. When I ran collectstatic locally, the project_images directory was created in the my local /static/media directory. But looking in the S3, there is no media directory and the *_images folders are at the bucket root. Do you know why this is?

As an aisde, I'm not using the thumbnail plugin ATM.

— Reply to this email directly or view it on GitHub https://github.com/MuckRock/muckrock/issues/467#issuecomment-119635244.