Skip to content

Rybbit: 最初のアカウントを作ったので登録を閉じる - #185

Merged
5ym merged 1 commit into
mainfrom
rybbit-close-signup
Sep 19, 2026
Merged

5ym merged 1 commit into
mainfrom
rybbit-close-signup

Conversation

@5ym

@5ym 5ym commented Sep 19, 2026

Copy link
Copy Markdown
Collaborator

rt.doany.io で管理者のアカウントを作ったので、DISABLE_SIGNUPNEXT_PUBLIC_DISABLE_SIGNUP を true に。false のままだと誰でもこのインスタンスにアカウントを作れる。README の手順にも済みと書いた。

🤖 Generated with Claude Code

https://claude.ai/code/session_01JgGYf5qbjDXYhtzcpLsGvv

@claude

claude Bot commented Sep 19, 2026

Copy link
Copy Markdown

Claude finished @5ym's task in 23s —— View job


レビュー結果

変更内容:

  • rybbit.yamlDISABLE_SIGNUP / NEXT_PUBLIC_DISABLE_SIGNUP"true" に変更
  • README に完了の経緯と補足(招待メールが Resend 未設定で届かない旨)を追記

確認したこと:

  • DISABLE_SIGNUP / NEXT_PUBLIC_DISABLE_SIGNUP はこの 2 箇所のみで使われており、両方とも一致して "true" に更新されている(片方だけ更新漏れという状態ではない)
  • README の手順 3 で予告されていた対応そのもので、コメントの更新内容も実態と整合している
  • シークレットや認証情報の混入なし。ロジック的な誤りや境界値の問題も無い(単純な文字列値の変更)

所感:
問題無しです。管理者アカウント作成後すぐに登録を閉じるという README の手順通りの、意図通りの変更です。

@5ym
5ym merged commit 5b4601c into main Sep 19, 2026
1 check passed
@5ym
5ym deleted the rybbit-close-signup branch September 19, 2026 14:03
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