Skip to content

Remove inline auth forms from UserMenu#25

Merged
xaelophone merged 1 commit into
mainfrom
fix/remove-inline-auth-forms
Feb 25, 2026
Merged

Remove inline auth forms from UserMenu#25
xaelophone merged 1 commit into
mainfrom
fix/remove-inline-auth-forms

Conversation

@xaelophone

Copy link
Copy Markdown
Collaborator

Summary

  • Remove duplicate login/signup/forgot-password inline forms from the UserMenu dropdown that bypassed invite-code gating
  • Replace with <Link> navigation to the existing /login and /signup dedicated pages
  • Remove ~234 lines of dead code (handlers, state, refs, form markup, unused supabase import)

Test plan

  • Click avatar when logged out — dropdown shows "Sign In" and "Sign Up" as links
  • Click "Sign In" — navigates to /login page, dropdown closes
  • Click "Sign Up" — navigates to /signup page (invite code flow), dropdown closes
  • Logged-in flows (change password, billing, MCP) still work unchanged
  • npm run web:build passes

🤖 Generated with Claude Code

The UserMenu dropdown had duplicate login/signup/forgot-password forms
that bypassed invite-code gating and used weaker auth calls. Replace
them with Link navigation to the existing /login and /signup pages.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vercel

vercel Bot commented Feb 25, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
hermes Ready Ready Preview, Comment Feb 25, 2026 9:09pm

Request Review

@xaelophone
xaelophone merged commit c1013ef into main Feb 25, 2026
8 checks passed
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