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

Fixes #488 - Adding env sample files #554

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jonespm
Copy link
Member

@jonespm jonespm commented Oct 22, 2024

Leaving as draft while a few more values are researched and tested.

@jonespm jonespm linked an issue Oct 22, 2024 that may be closed by this pull request
@@ -0,0 +1,22 @@
#DEBUG=False
Copy link
Member

Choose a reason for hiding this comment

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

There are a couple of environment attributes defined in the docker-compose.yaml file, inside both web and database services. Can those be moved into the .env too?

Copy link
Member

Choose a reason for hiding this comment

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

Good question. The master branch currently has…

      - ALLOWED_HOSTS=*
      - DEBUG=True
      - DATABASE_URL=postgresql://admin:admin_pw@database/admin
      - [email protected]

As I was working with the app in local dev., I had to set some values there for running the Twilio tests. I was thinking at the time that they should go in a config file as Matt does here. I think the values listed above could also go there.

Comment on lines 10 to 12
- DEBUG=True
- DATABASE_URL=postgresql://admin:admin_pw@database/admin
- [email protected]
Copy link
Member

Choose a reason for hiding this comment

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

As suggested in earlier comments, let's move these to .env, too.

Copy link
Member Author

Choose a reason for hiding this comment

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

Sure we can move these, but I think having these might require that a .env file exists unless these are in the code. That's probably fine as a practice. I'd also update the README, I was still testing this to make sure there was nothing else and having it as draft.

This would only affect development but is really useful to have for development.

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

Successfully merging this pull request may close these issues.

Implement environment file configs (and update documentation)
3 participants