-
Notifications
You must be signed in to change notification settings - Fork 62
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
Add setting to exclude directories from STATIC_ROOT #124
Conversation
I wonder if there's way to handle this upstream of bakery. Could you remove it from our INSTALLED_APPS? Or change your staticfiles configuration elsewhere in your settings? |
I am closing this PR pending a response to my question above. |
I forgot about this PR. To answer your question: Removing Also, when running a build, this will exclude the directory from the Finally, this doesn't work for e.g. |
Reopening for discussion. Is there a use case here beyond the admin app? I wonder if we might be better off tailoring a fix specifically for the admin, though I'm not fully swayed that this static file deployment of admin files is a very big deal. |
I think using wagtail (or any other CMS that includes it's own static files) is a reasonable use case. Currently these folders are included in the bakery output:
So, in my project I'm using:
More people are having this issue, hence this related issue in wagtail-bakery. |
Could we solve this further upstream with how we handle collectstatic's configuration, rather than having them be collected only to be clobbered? |
This allows to exclude e.g. the /admin/ directory which is not required in the static site.
See also #107.