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

Project initialization environment improvements #175

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

Conversation

ryangallen
Copy link

  • Update default cookiecutter values; use random password/secret values
  • Store environment secrets in .env file
  • Automatically initialize git repository
  • Add .gitattributes and .gitignore files
  • Add cookiecutter links to project README
  • Remove application secrets from source code; read from environment variables
  • Change user crud get functions to use one_or_many()
  • Fix frontend errors by casting to String

@Buuntu
Copy link
Owner

Buuntu commented Nov 3, 2021

This is a lot of changes for a pull request. Can you open a discussion first and link to it? Some of these seem like good ideas but I'd rather approve them individually after some thought instead of in one go. This is a lot for a single pull request imo

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

2 participants