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

Removes naomi from debug mode #336

Merged
merged 1 commit into from Jun 26, 2020
Merged

Conversation

caiocarrara
Copy link
Contributor

The commit b935e08 replaced naomi with mailhog and it seems the naomi
app was kept in INSTALLED_APPS declaration when DEBUG == True. Since in
the referred commit the naomi dependency was removed from requirements
file, the project is currently failing to run in debug mode in a fresh
install.

This change removes the naomi app from INSTALLED_APPS when DEBUG ==
True and removes the django-naomi dependency.

The commit b935e08 replaced naomi with mailhog and it seems the naomi
app was kept in INSTALLED_APPS declaration when DEBUG == True. Since in
the referred commit the naomi dependency was removed from requirements
file, the project is currently failing to run in debug mode in a fresh
install.

This change removes the naomi app from INSTALLED_APPS when DEBUG ==
True and removes the django-naomi dependency.

Signed-off-by: Caio Carrara <eu@caiocarrara.com.br>
Copy link
Collaborator

@berinhard berinhard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Simple enough 👍

@turicas turicas merged commit a26e6ee into turicas:develop Jun 26, 2020
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.

None yet

3 participants