Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants