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

moving loading of local.env to manage.py #57

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

dave-leblanc
Copy link

@dave-leblanc dave-leblanc commented Apr 26, 2016

So here's what was happening. New Django project from latest edge.

when deploying to the server because manage.py was looking for an ENV var to define the settings location without ever loading the local.env it was always looking for the development settings because that's what was defaulted in manage.py

subsequently this applied to wsgi.py which was defaulting to production. If a users local.env ever called for a different settings file it would not be loaded.

Moving the loading of the local.env file into the environment at the manage.py and wsgi.py levels means that it is no longer needed in settings.base

@dave-leblanc
Copy link
Author

I've added an additional change to the gitignore as the /site/ dir being created inside project_dir was showing as an untracked change on my deployment server.

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 this pull request may close these issues.

2 participants