Skip to content

Conversation

@dborowiecki
Copy link
Contributor

TICKET: OPL-83

@qgerome
Copy link
Contributor

qgerome commented Mar 31, 2022

Please, do not merge that. It's too complex and I can't see what could justify to use such system instead of adding a block of text in the modules' readme explaining what are the settings that need to be added/updated in the settings.py.

You could also use another settings.mycase.py file overriding the default one with your custom configuration using the DJANGO_SETTINGS_MODULE env variable. https://docs.djangoproject.com/en/4.0/topics/settings/#designating-the-settings

IMHO, customising the settings.py is not the responsibility of the modules but the developer/implementer. (and it's a one-time operation)

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