Skip to content

Bugfix/484/enable single sign on for react app through people depot #485

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 44 commits into
base: main
Choose a base branch
from
Open

Bugfix/484/enable single sign on for react app through people depot #485

wants to merge 44 commits into from

Conversation

ethanstrominger
Copy link
Member

Fixes #484

What changes did you make?

  • update-table.md : formatting changes
  • .env.docker-example
    • Add CORS_ALLOWED_ORIGINS
    • Remove all settings related to local development
  • Dockerfile - updated graphiz version
  • views.py: add patch method for user profile to be used. by React app
  • settings.py
    • Added corsheaders to apps
    • Added CORS_ALLOWED_ORIGINS, CORS_ALLOW_CREDENtIALS, and CORS_ALLOW_HEADERS
    • Added CORS to middleware
    • Changed core.utils.jwt to core.utils.jwt_handler to disambiguate from the cors package
    • Added django-cors-headers package
  • requirements.txt: added django-cors-headers

Why did you make the changes?

  • make changes so you can call from REACT

Testing

ethanstrominger and others added 30 commits October 16, 2024 08:10
@ethanstrominger
Enable CORS for React application
582dfde
@ethanstrominger
Partially working, some code commented out
dd3fc1c
@ethanstrominger
Handle jwt decode error,
4627e3f
@ethanstrominger
WIP
2553707
@ethanstrominger
Small changes
0c52805
@ethanstrominger
Resolve views.py conflict
e94bfd6
@ethanstrominger
Merge branch 'main' of https://github.com/hackforla/peopledepot into …
83082c8 
…cors
@ethanstrominger
Change post to patch
884e8d4
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
319ae56 
for more information, see https://pre-commit.ci
@ethanstrominger
Changes to support React
72f72dc
@ethanstrominger
Changes for CORS so REACT works
e1de3dc
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
3e96aaf 
for more information, see https://pre-commit.ci
@ethanstrominger
Add blank line
1c31653
@ethanstrominger
Merge branch 'cors' of https://github.com/ethanstrominger/peopledepot …
dd4036b 
…into cors
@ethanstrominger
Revert changes to files related to /login local
6b233c9
@ethanstrominger
Merge branch 'cors' of https://github.com/ethanstrominger/peopledepot …
698a3eb 
…into cors
@ethanstrominger
Remove /login API implementation
dc5b878
@ethanstrominger
Restore jwt.py
dddcb7c
@ethanstrominger
Restore update-table.md
ace3579
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
368e92a 
for more information, see https://pre-commit.ci
@ethanstrominger
Restore jwt.py
6d69f40
@ethanstrominger
Merge branch 'cors' of https://github.com/ethanstrominger/peopledepot …
f6e880f 
…into cors
@ethanstrominger
Remove duplicate env variable in .env.docker-example
ac7bb08
@ethanstrominger
Restore docker-compose.yml
9cc11eb
@ethanstrominger
Remove unused import
ddfe984
@ethanstrominger
pre-commit and update-table.md
ed177de
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
1909194 
for more information, see https://pre-commit.ci
@ethanstrominger
Try to fix update-table.md hanging space
8e1f7db
@ethanstrominger
Merge branch 'cors' of https://github.com/ethanstrominger/peopledepot …
c18fe5d 
…into cors
@ethanstrominger
Fix trailing whitespace
c75db16
@ethanstrominger
Modify to tricker pre-commit
775d753
@ethanstrominger
Update graphviz
0d70d8f
@ethanstrominger
Squashed commit of the following:
8d9bd82 
commit 0d70d8f
Author: Ethan Strominger <[email protected]>
Date:   Wed Dec 18 11:42:21 2024 -0500

    Update graphviz

commit 775d753
Author: Ethan Strominger <[email protected]>
Date:   Sun Dec 1 17:25:56 2024 -0500

    Modify to tricker pre-commit

commit c75db16
Author: Ethan Strominger <[email protected]>
Date:   Sun Dec 1 17:23:16 2024 -0500

    Fix trailing whitespace
@ethanstrominger
Merge branch 'cors' of https://github.com/ethanstrominger/peopledepot …
8a73893 
…into cors
@ethanstrominger
Squashed commit of the following:
2924fb9 
commit 0d70d8f
Author: Ethan Strominger <[email protected]>
Date:   Wed Dec 18 11:42:21 2024 -0500

    Update graphviz

commit 775d753
Author: Ethan Strominger <[email protected]>
Date:   Sun Dec 1 17:25:56 2024 -0500

    Modify to tricker pre-commit

commit c75db16
Author: Ethan Strominger <[email protected]>
Date:   Sun Dec 1 17:23:16 2024 -0500

    Fix trailing whitespace
@ethanstrominger
Merge branch 'main' of https://github.com/hackforla/peopledepot
302df99
@ethanstrominger
Merge main from pd
e1fa593
@github-project-automation github-project-automation bot moved this to PR Needs review (automated column, do not place items here manually) in P: PD: Project Board Mar 18, 2025
@github-project-automation github-project-automation bot moved this from PR Needs review (automated column, do not place items here manually) to ✅Done in P: PD: Project Board Mar 18, 2025
@github-project-automation github-project-automation bot moved this from ✅Done to PR Needs review (automated column, do not place items here manually) in P: PD: Project Board Mar 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
P: PD: Project Board
Status: PR Needs review (automated column, do not place items here manually)
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Enable single sign on for React app through People Depot
1 participant
@ethanstrominger