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

Bdl add auth #27

Merged
merged 3 commits into from
May 17, 2023
Merged

Bdl add auth #27

merged 3 commits into from
May 17, 2023

Conversation

bdleavitt
Copy link
Owner

Purpose

Addresses #10
Makes the authenticated user details available to the frontend app via frontend/src/auth/authUtils.tsx and for the backend via
backend/auth_utils.py (from auth.auth_utils import get_authenticated_user_details)

Does this introduce a breaking change?

[ ] Yes
[x ] No

Pull Request Type

What kind of change does this Pull Request introduce?

[ ] Bugfix
[ x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

Local dev
can test this by running the app locally.

  • Clone the repo.
  • open two terminals.
  • Frontend
    • Navigate to frontend folder
    • npm run dev
  • Backend
  • *Navigate to app folder (not backend)
    • Run start.ps1

** Deploy to app service**
Deploy to app service. Then run it.

  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

@bdleavitt bdleavitt requested a review from kjpap as a code owner May 17, 2023 23:53
@bdleavitt bdleavitt merged commit b760921 into main May 17, 2023
@bdleavitt bdleavitt deleted the bdl-add-auth branch May 17, 2023 23:55
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.

1 participant