search

adicu / adi-website

[DEPRECATED] The website for the Application Development Initiative, built on Eventum.
https://adicu.com
11 stars 15 forks source link

Accept 851x315 px media #274

Open parthibanloganathan opened 8 years ago

parthibanloganathan commented 8 years ago

That's the size of FB cover images. It's currently not accepted (I understand why you wouldn't) but given that we design our media for FB, we should also adopt that size for Eventum events. @danrschlosser

schlosser commented 8 years ago

Hmm, that really shouldn't be too small at all, there must be another problem happening. what kind of problem are you getting?

parthibanloganathan commented 8 years ago

Got a 413 Request entity too large. Uploaded a smaller image and worked like a charm. Didn't look into it too much so might be misdiagnosing the problem. Seemed like it was an issue with image size.

See this for relevant info. May need to reconfigure Nginx.

eunicekokor commented 8 years ago

I don't think it's a physical size thing, it might be a file size thing? 413 is the default flask error for uploading any file is > the default maximum of 1024? bytes.. in our config it says our max size is 256, is this true--> https://github.com/adicu/adi-website/blob/master/config/flask_config.py#L32

On Wed, Mar 2, 2016 at 8:14 PM, Parthiban Loganathan < notifications@github.com> wrote:

Got a 413 Request entity too large. Uploaded a smaller image and worked like a charm. Didn't look into it too much so might be misdiagnosing the problem. Seemed like it was an issue with image size.

— Reply to this email directly or view it on GitHub https://github.com/adicu/adi-website/issues/274#issuecomment-191523914.

schlosser commented 8 years ago

That's the size of the log file, not file uploads. By default, Flask doesn't limit the file size, although it can be set. I think it actually might be a problem with NGINX having a max upload size (default is 1.5MB I think). So we should fix that.

But also @parthibanloganathan you could probably "export for web" or whatever and you should be able to get under 1.5MB, I think...

Either way we should fix on NGINX