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
Added Django Skeleton #35
Conversation
It is working, but only when settings is left with its vars. Ex: I've tried to remove SECRET_KEY from settings.py and got: raise ImproperlyConfigured("The SECRET_KEY setting must not be empty.") django.core.exceptions.ImproperlyConfigured: The SECRET_KEY setting must not be empty.
It is working, but only when settings is left with its vars. Ex: I've tried to remove SECRET_KEY from settings.py and got: raise ImproperlyConfigured("The SECRET_KEY setting must not be empty.") django.core.exceptions.ImproperlyConfigured: The SECRET_KEY setting must not be empty.
2dfaac9
to
e040b04
Compare
@@ -1,5 +1,5 @@ | |||
from django.apps import AppConfig | |||
|
|||
|
|||
class BarConfig(AppConfig): | |||
class DJangoDynaconfig(AppConfig): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we have a little typo in the class name
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tried I little merge with Dynaconf with Config, but maybe I've failed ;)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i think @mazulo is refering to "DJango..." instead of "Django"
It is working, but only when settings is left with its vars.
Ex: I've tried to remove SECRET_KEY from settings.py and got:
raise ImproperlyConfigured("The SECRET_KEY setting must not be empty.")
django.core.exceptions.ImproperlyConfigured: The SECRET_KEY setting must not be empty.
Now I got what you mean we need to keep standard DJANGO var on settings.py, only to please the health check of the project. On your second solution, patching manage.py and wsgi.py are the var also necessary on settings.py? If not, I would go with the second solution ;)