Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat: log out and redirect user when converting to oidc #8347

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
Merged
merged 3 commits into from
Jul 10, 2023
Merged

feat: log out and redirect user when converting to oidc #8347

merged 3 commits into from
Jul 10, 2023

Conversation

Emyrk
Copy link
Member

@Emyrk Emyrk commented Jul 6, 2023 •
edited
Loading

Decision from here: #8327

Log out user and redirect to login page and log out user when
they convert to oidc.

Screenshot from 2023-07-06 12-31-43

Emyrk added 2 commits July 6, 2023 12:29
@Emyrk
feat: log out user on conver to oidc
596605e 
Log out user and redirect to login page and log out user when
they convert to oidc.
@Emyrk
Update message
0293d70
@Emyrk Emyrk changed the title feat: log out user on conver to oidc feat: log out and redirect user when converting to oidc Jul 6, 2023
@Emyrk Emyrk requested a review from deansheather July 6, 2023 18:05
deansheather
deansheather approved these changes Jul 10, 2023
View reviewed changes
Copy link
Member

@deansheather deansheather left a comment

Choose a reason for hiding this comment

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

Backend LGTM

@Emyrk Emyrk merged commit 2ee406d into main Jul 10, 2023
@Emyrk Emyrk deleted the stevenmasley/oidc_out_exp branch July 10, 2023 14:25
@github-actions github-actions bot locked and limited conversation to collaborators Jul 10, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@deansheather deansheather deansheather approved these changes

Assignees

@Emyrk Emyrk

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Emyrk @deansheather