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

Cors #451

Draft
wants to merge 29 commits into
base: main
Choose a base branch
from
Draft

Cors #451

wants to merge 29 commits into from

Conversation

ethanstrominger
Copy link
Member

@ethanstrominger ethanstrominger commented Nov 30, 2024

Fixes #replace_this_text_with_the_issue_number

What changes did you make?

  • update-table.md : minor change from .pre-commit

Following files were changed to implement /login API so that developers can test the API without connecting to COGNITO

  • custom_api.py
  • urls.py
  • views.py: add patch method to user
  • settings.py: change Move JWT setting to be conditional based on COGNITO variables being set. See also CORS changes

CORS changes

  • settings.py
  • .env.docker-example
  • requirements.txt

Build related changes:

  • buildrun.sh: remove -d flag to make it easier to find out why Django API Server returns an error
  • remove echo DJANGO_SUPERUSER as sometimes this would make script fail
  • docker-compose.yml: avoid running mkdocs by setting env variable

Why did you make the changes?

  • make changes so you can call from REACT
  • Add a login URL for developer so you do not need to connect to COGNITO

ethanstrominger and others added 29 commits October 16, 2024 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: PR Needs review (automated column, do not place items here manually)
Development

Successfully merging this pull request may close these issues.

1 participant