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.

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

PB-17h: Logout user #32

Merged
merged 3 commits into from
Sep 27, 2023
Merged

PB-17h: Logout user #32

merged 3 commits into from
Sep 27, 2023

Conversation

stianjsu
Copy link

image

Implements logout endpoint and log out button in the frontend

@annabjorgo
Copy link

Jeg får 404 feilmelding når jeg prøver å logge ut, har det skjedd med deg før?

@stianjsu
Copy link
Author

stianjsu commented Sep 12, 2023 •
edited
Loading

Jeg får 404 feilmelding når jeg prøver å logge ut, har det skjedd med deg før?

Hmmm, nei har ikke det. Skjer det alltid?

sergiosja
sergiosja approved these changes Sep 12, 2023
View reviewed changes
Copy link
Collaborator

@sergiosja sergiosja left a comment

Choose a reason for hiding this comment

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

Fint og bra! Alt ser ut til å funke her. Ingen 404 og cookies forsvinner når jeg logger ut. Bra jobba:)

@stianjsu stianjsu requested a review from annabjorgo September 19, 2023 08:44
annabjorgo
annabjorgo approved these changes Sep 25, 2023
View reviewed changes
Copy link

@annabjorgo annabjorgo left a comment

Choose a reason for hiding this comment

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

Det funket for meg nå, bra jobba, bare å merge!

@stianjsu stianjsu merged commit c54f0e2 into main Sep 27, 2023
@stianjsu stianjsu deleted the logout-user branch September 27, 2023 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sergiosja sergiosja sergiosja approved these changes

@annabjorgo annabjorgo annabjorgo approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@stianjsu @annabjorgo @sergiosja