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

Check config up to date #27

Open
martinzlocha opened this issue Jul 15, 2017 · 2 comments
Open

Check config up to date #27

martinzlocha opened this issue Jul 15, 2017 · 2 comments

Comments

@martinzlocha
Copy link
Member

Detailed Description

Check whether the config.py contains the same fields as the sampleconfig.py

Context

When the sampleconfig is updated the config file should be changed as well to match this. If this is not done it will lead to errors.

Possible Implementation

Either add a version field to config and check if they are identical or check whether there are the same variables in config and sampleconfig. Second option would be nicer because it would not involve changing version numbers. The check should be implemented in manage.py given that its the entry point for the server.

@levex
Copy link
Member

levex commented Jul 15, 2017

I’m not sure I follow. Why is this needed? Wouldn’t Django automatically exit saying that a variable is missing from config.py?

@martinzlocha
Copy link
Member Author

Thought it would be nicer to get a message telling you that config is outdated rather than a long error which could confuse people but not sure how useful it is.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants