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

add logout flow #44

Merged
merged 2 commits into from
Dec 11, 2024
Merged

add logout flow #44

merged 2 commits into from
Dec 11, 2024

Conversation

wkim10
Copy link
Collaborator

@wkim10 wkim10 commented Dec 11, 2024

Description

Made logout button on SideNavBar functional

Issues

None :)

Screenshots

None :)

Test

Try signing in as different users and clicking "logout"

Possible Downsides

None :)

Additional Documentations

None :)

@wkim10 wkim10 requested a review from johnny-t06 December 11, 2024 06:42
Copy link
Collaborator

@johnny-t06 johnny-t06 left a comment

Choose a reason for hiding this comment

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

LGTM, looks great Won. I just lowercased the path.

@johnny-t06 johnny-t06 merged commit f0006d7 into main Dec 11, 2024
1 check passed
@johnny-t06 johnny-t06 deleted the won/logout-flow branch December 11, 2024 06:54
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.

2 participants