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

Company logo disappears on company update #28

Open
robinchew opened this issue Sep 5, 2016 · 10 comments
Open

Company logo disappears on company update #28

robinchew opened this issue Sep 5, 2016 · 10 comments

Comments

@robinchew
Copy link
Collaborator

Happens in production.

@Mause
Copy link
Contributor

Mause commented Sep 5, 2016

Probably something to do with form.cleaned_data vs request.FILES in create vs update

@mattharley
Copy link
Owner

could be because we aren't uploading to S3 - it's only uploading to the local file system on Heroku which is flushed every time the dyno restarts. i can setup the S3 credentials if someone's happy to implement django-storages S3: https://django-storages.readthedocs.io/en/latest/backends/amazon-S3.html

@Mause
Copy link
Contributor

Mause commented Sep 6, 2016

Does it still happen after those changes I made last night?

@robinchew
Copy link
Collaborator Author

Sorry I haven't deployed yet. You wanna give Mr. Mause deployment permission @mattharley ?

@robinchew
Copy link
Collaborator Author

Just deployed. I think there are 2 problems here.

  1. Just pressing company update button would delete the logo. Now fixed after deploy.
  2. What mattharley's talking about.

@mattharley
Copy link
Owner

@Mause can you please sign up to Heroku and let me know the email you used so I can then add you as a contributor to the app

Whoever wants to tackle the Amazon S3 upload thingy (shouldn't be too difficult with the examples on the django-storages site) please email and I'll share the access keys with you.

I'm on matt@mattharley.com

@Mause
Copy link
Contributor

Mause commented Sep 7, 2016

I already have a Heroku account under my old psuedonym email address, jack.thatch@gmail.com

@mattharley
Copy link
Owner

cool added you now

On 7 September 2016 at 22:30, Dominic R. May notifications@github.com
wrote:

I already have a Heroku account under my old psuedonym email address,
jack.thatch@gmail.com


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#28 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AArt9ItsVSEkn37Su-5ritsCiUMSwR3jks5qnsqcgaJpZM4J1Ott
.

Matt Harley

matt@mattharley.com
mobile: +61 421 933 877
skype: matt.harley

Currently I'm the chief tech guy at hipflask
Website/blog: http://www.hipflaskapp.com/ http://www.hipflaskapp.com/
iOS: http://apple.co/12XnhZn
Android: http://bit.ly/1plwrbP

@bendog
Copy link
Collaborator

bendog commented Jul 30, 2018

has this been resolved? should some of these old issues be closed?

@robinchew
Copy link
Collaborator Author

Well since there is no company page anymore. I guess?

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

4 participants