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

Set up django #11

Merged
merged 3 commits into from
Feb 2, 2025
Merged

Set up django #11

merged 3 commits into from
Feb 2, 2025

Conversation

whlpatricia
Copy link
Collaborator

Set up django for the project. Run pip install -r requirements.txt to install all dependencies (which may be added or reduced in the future).

requirements.txt Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

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

move this inside the backend folder

# See https://docs.djangoproject.com/en/5.1/howto/deployment/checklist/

# SECURITY WARNING: keep the secret key used in production secret!
SECRET_KEY = 'django-insecure-sov_ozhr4w7jm7($tlge+)eu6nrpg5xuatp$8n91j4pk*p2vd$'
Copy link
Collaborator

Choose a reason for hiding this comment

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

Might be good to not leave this exposed esp if this is going to be deployed

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

oops i can add this to our github secrets then

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

oh i dont have perms for gh secrets, i'll just leave a note in the chat, we will have to figure out how to run it in CI in the future

Copy link
Collaborator

@VamsiGajjela VamsiGajjela left a comment

Choose a reason for hiding this comment

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

LGTM , move the file and this is fine

@whlpatricia whlpatricia merged commit be9ef79 into main Feb 2, 2025
1 check passed
@whlpatricia whlpatricia deleted the patricia-15-django-setup branch February 2, 2025 05:33
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.

2 participants