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

Show UserButton when user is logged in #240

Merged
merged 3 commits into from
Feb 25, 2025
Merged

Conversation

armans-code
Copy link
Collaborator

@armans-code armans-code commented Feb 19, 2025

(#202)

This basically just validates the user session on the backend using the auth cookie whenever AuthContext is rendered.

Not sure if this is the best method, let me know if we should approach this another way!

Screen.Recording.2025-02-18.at.8.41.17.PM.mov

Copy link

vercel bot commented Feb 19, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
uf-sase-website ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 25, 2025 10:17pm

@TheRickyZhang
Copy link
Collaborator

LGTM, I'm not too knowledgable about this but seems like a reasonable way

@ssihala
Copy link
Collaborator

ssihala commented Feb 19, 2025

Backend route looks good

@TheRickyZhang TheRickyZhang merged commit 4e94d94 into main Feb 25, 2025
6 checks passed
@TheRickyZhang TheRickyZhang deleted the arman/user-button branch February 25, 2025 22:18
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.

3 participants