Skip to content

Adjust frontend auth behavior#24

Merged
SuaYoo merged 6 commits intomainfrom
frontend-auth
Nov 24, 2021
Merged

Adjust frontend auth behavior#24
SuaYoo merged 6 commits intomainfrom
frontend-auth

Conversation

@SuaYoo
Copy link
Copy Markdown
Member

@SuaYoo SuaYoo commented Nov 23, 2021

Behavioral changes:

  • Redirect to /my-account when visiting /log-in if already logged in
  • Differentiate between unauthenticated API calls and page views (TODO: possibly introduce refresh tokens Handle expired JWT #22)

Added more unit tests as well.

Testing

To manually test, start the web app and go to /log-in. If you're already logged in, you should be redirected to /my-account. If you're not logged in, log in and attempt to go to /log-in to test.

@SuaYoo SuaYoo requested a review from ikreymer November 23, 2021 01:41
@SuaYoo SuaYoo self-assigned this Nov 23, 2021
@SuaYoo SuaYoo merged commit 04fbe6f into main Nov 24, 2021
@SuaYoo SuaYoo deleted the frontend-auth branch November 24, 2021 00:57
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