Skip to content

feat(login): 放出 Discord 登录按钮(灰度)+ 回调错误提示 - #389

Merged
longsizhuo merged 2 commits into
mainfrom
feat/discord-canary
Jul 24, 2026
Merged

feat(login): 放出 Discord 登录按钮(灰度)+ 回调错误提示#389
longsizhuo merged 2 commits into
mainfrom
feat/discord-canary

Conversation

@longsizhuo

Copy link
Copy Markdown
Member

配套后端 involutionhell-backend#53 的 Discord 灰度。

  • 重新放出登录页 Discord 按钮(此前因 OTP wiring 未完成而下线)。后端按 Discord id 白名单放行:名单内正常登入,名单外回调弹回 /login?error=discord_canary
  • 新增 LoginErrorNotice:读 ?error= 显示提示(discord_canary → 灰度中;其它 → 通用重试)。用客户端 useSearchParams + <Suspense>,登录页保持 SSG(build 表仍 ● /[locale]/login)。
  • i18n:login.errorDiscordCanary / login.errorGeneric(zh/en)。

验证

pnpm typecheck / pnpm lint(0 error)/ pnpm test(67 通过)/ pnpm build(login 仍 ● SSG)。

⚠️ 合并顺序:先等后端 #53 部署上线(闸生效),再合本 PR 放按钮,避免出现 Discord 对所有人开放的窗口。

🤖 Generated with Claude Code

https://claude.ai/code/session_01RtSNVtrEkCsFVHrbt4Zjgi

github-actions Bot added 2 commits July 24, 2026 13:55
后端已按 Discord id 白名单放行(auth.discord.allowlist),前端把 Discord 按钮
重新放出:名单内的人正常登入,名单外的人回调被弹回 /login?error=discord_canary。

登录页新增 LoginErrorNotice:读 ?error= 显示提示(discord_canary → "灰度中",
其它 → 通用重试)。用客户端 useSearchParams + <Suspense>,登录页保持 SSG
(build 表仍 ● /[locale]/login,未退回 ƒ)。
Copilot AI review requested due to automatic review settings July 24, 2026 16:41

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@vercel

vercel Bot commented Jul 24, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
involutionhell-github-io Ready Ready Preview, Comment Jul 24, 2026 4:55pm
website-preview Ready Ready Preview, Comment Jul 24, 2026 4:55pm

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