feat(app): switch to better-auth#696
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Thank you for following the naming conventions! 🙏 |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1a58d2e5b8
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| ? `/auth/${props.provider}?redirect=${encodeURIComponent(props.redirectUrl)}` | ||
| : `/auth/${props.provider}` | ||
| window.location.href = url | ||
| signIn.social({ provider: props.provider }) |
There was a problem hiding this comment.
Preserve redirect target in social sign-in
This change drops the redirect parameter when starting OAuth (signIn.social({ provider })), so users who were sent to /login?redirect=... after hitting a protected page will no longer be returned to their original destination after social auth. The login flow still passes redirect-url into this component, but the prop is no longer read, which is a user-visible regression in navigation behavior.
Useful? React with 👍 / 👎.
commit: |
…ation # Conflicts: # pnpm-lock.yaml
…ation # Conflicts: # apps/shelve/package.json # pnpm-lock.yaml
Scope: PR #696 auth DX pass (PR files + auth neighbors). No remaining direct signIn/client OTP calls or explicit ~/utils/auth-error imports in PR files.
Auth migration to Better Auth.
@onmax/nuxt-better-auth/better-auth.main.Testing:
pnpm typecheck.Notes:
BETTER_AUTH_SECRET; production still should set it.HUB_APPLY_MIGRATIONS_DURING_BUILD=trueto enable).Review Lanes
Review this PR in this order:
Detailed map: docs/adrs/auth-migration-review-map.md