Skip to content
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

CityInfo page not saving images #146

Open
Jcrowley opened this issue Feb 10, 2015 · 7 comments
Open

CityInfo page not saving images #146

Jcrowley opened this issue Feb 10, 2015 · 7 comments

Comments

@Jcrowley
Copy link

Set up City72 on Heroku and development box (MacOSX10.10). CityInfo saves text, but not images. The database query shows null values for img_url and resident_image_url despite having set files for upload (see image). Happens on Heroku and localhost. Note: I changed email and client url.

Processing by Cms::CitiesController#update as HTML
  Parameters: {"city"=>"{\"id\":1,\"name\":\"TIDES\",\"abbreviation\":\"TIDE\",\"moniker\":\"staff\",\"possessive\":\"TIDES's\",\"color_cd\":0,\"brought_by\":\"TIDES\",\"agency_url\":\"client.net\",\"contact_email\":\"scrubbed@nope.com\",\"statement\":null,\"image_url\":null,\"resident_image_url\":null,\"affiliates\":[],\"affiliates_attributes\":[]}", "id"=>"1"}
  Mode Load (0.3ms)  SELECT "modes".* FROM "modes" LIMIT 1
  City Load (0.3ms)  SELECT "cities".* FROM "cities" LIMIT 1
  About Load (0.3ms)  SELECT "abouts".* FROM "abouts" LIMIT 1
  Analytic Load (0.3ms)  SELECT "analytics".* FROM "analytics" LIMIT 1
  AdminUser Load (0.4ms)  SELECT "admin_users".* FROM "admin_users" WHERE "admin_users"."id" = 1 LIMIT 1
   (0.1ms)  BEGIN
   (0.4ms)  UPDATE "cities" SET "name" = 'TIDES', "abbreviation" = 'TIDE', "possessive" = 'TIDES''s', "brought_by" = 'TIDES', "updated_at" = '2015-02-10 03:39:32.223122' WHERE "cities"."id" = 1
   (0.8ms)  COMMIT
  Affiliate Load (0.3ms)  SELECT "affiliates".* FROM "affiliates" WHERE "affiliates"."city_id" = 1

screenshot 2015-02-09 22 43 45

@arena
Copy link
Collaborator

arena commented Feb 10, 2015

I've never tried to upload images to a local server. I've only done it after deploying. I think this bug could be limited to just local environments. Please note if the problem persists after you deploy.

Thanks for filing the bug!

@Jcrowley
Copy link
Author

It happens on Heroku as well. It happened there first, which is why I installed locally to see why. Thanks!

This message was handcrafted by a big thumb pressing letters on a tiny keyboard.

On Feb 10, 2015, at 12:12 AM, Arena notifications@github.com wrote:

I've never tried to upload images to a local server. I've only done it after deploying. I think this bug could be limited to just local environments. Please note if the problem persists after you deploy.

Thanks for filing the bug!


Reply to this email directly or view it on GitHub.

@arena
Copy link
Collaborator

arena commented Feb 13, 2015

Can you share your heroku instance? I'd be really curious to reproduce this.

Thanks!

On Mon, Feb 9, 2015 at 9:43 PM, John Crowley notifications@github.com
wrote:

It happens on Heroku as well. It happened there first, which is why I
installed locally to see why. Thanks!

This message was handcrafted by a big thumb pressing letters on a tiny
keyboard.

On Feb 10, 2015, at 12:12 AM, Arena notifications@github.com wrote:

I've never tried to upload images to a local server. I've only done it
after deploying. I think this bug could be limited to just local
environments. Please note if the problem persists after you deploy.

Thanks for filing the bug!


Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHub
#146 (comment).

@arena
Copy link
Collaborator

arena commented Jul 13, 2015

Can somebody please try to reproduce this and then write steps to reproduce?
(I suspect you need a large image to get this to reproduce.)

@rkcheung
Copy link

Is there a recommended size/resolution for these images? I am unable to get anything to show up at all.

@arena
Copy link
Collaborator

arena commented May 20, 2016

@rkcheung I found the CMS to be buggy. I'm currently working on a derivative of this project that doesn't rely on the CMS. Depending on how technical you are, you might want to fork https://github.com/gnv-org/gnv instead. If you want to stick to using the CMS, image size (file size) is definitely a factor, but the whole implementation was complicated by the Cloudinary integration, cache time, and the idea of using it as a user-friendly uploader and also a CDN---it's hard to do both things well at all times.

@rkcheung
Copy link

@arena Thanks - I will try yours.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants