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

collectstatic with S3BotoStorage #71

Closed
erm0l0v opened this issue Dec 15, 2015 · 1 comment · Fixed by #72
Closed

collectstatic with S3BotoStorage #71

erm0l0v opened this issue Dec 15, 2015 · 1 comment · Fixed by #72

Comments

@erm0l0v
Copy link
Contributor

erm0l0v commented Dec 15, 2015

Hi Guys.

I have problem with collectstatic command.

I use S3BotoStorage with ManifestFilesMixin as a static files storage and I got an error after run collectstatic.

https://github.com/gregmuellegger/django-sortedm2m/blob/1.1.1/sortedm2m/forms.py#L28

static function tries to get file from S3, but it impossible until collectstatic will be done.

@erm0l0v erm0l0v changed the title collectstatic with StaticRootS3BotoStorage collectstatic with S3BotoStorage Dec 15, 2015
gregmuellegger added a commit that referenced this issue Dec 17, 2015
Remove use of static function in form media

Fixes #71.
@gregmuellegger
Copy link
Collaborator

Yep, that makes sense. Thanks. I didn't know that the URL will be prefixed with the STATIC_URL setting :)

gregmuellegger added a commit that referenced this issue Dec 17, 2015
jlemaes added a commit to jlemaes/django-sortedm2m that referenced this issue Jan 9, 2016
* commit '476fbb59297e05fd21b0f8114eeabbcea8998604':
  Changelog for jazzband#71 and jazzband#72
  SortedCheckboxSelectMultiple remove static
  Remove branch whitelisting for travis
  Try container based builds on travis
  Bump version to 1.1.1.dev1
  Prepare 1.1.1 release
  Add PyPI badge to README
  Fix collapsing filter bar for the new Django 1.9 admin.
  Bump version to 1.1.0.dev1
  Preparing 1.1.0 release
  Mention jazzband#68 in changelog
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

Successfully merging a pull request may close this issue.

2 participants