同一人物が複数回登録するのを防止。

[CaffeeLake]

現状アカウントやメールアドレスが異なれば、同一人物でも複数個アカウントを作ることができる(実際に作っている人がいる)。
そのため、学籍番号・電話番号等も一致した場合弾く必要がある。

[SnakeSneakS]

関連箇所メモ:

• https://github.com/MISW/portal/blob/main/backend/schema/users.sql#L17
• https://github.com/MISW/portal/blob/main/backend/usecase/session.go#L98
• DB側で ALTER TABLE users ADD CONSTRAINT uq_users_student_id UNIQUE(student_id); 的な何かのSQL

[CaffeeLake]

ソースコード上でUNIQUEをつけた場合、今あるデータベースの中身とConflictが起きると思うけど、どうなるのかしら。

[SnakeSneakS]

あーdockerfile見てみたら自動でマイグレーション実行しそう(のオプションついてそう)なので、別でsql実行する必要ないかも...
https://github.com/MISW/portal/blob/main/backend/docker-entrypoint.sh#L64
https://github.com/MISW/portal/blob/main/backend.dockerfile#L49

起動時にDBスキーマ書き変わってそれで終わりな気がする..？(データベースの中身がUNIQUEにできる状態だったら)

[CaffeeLake]

データベースの中身がUNIQUEできないからどうなるんだろうと。

[SnakeSneakS]

そっ、それは消せばいいのでは..？　(或いは当人困惑するかもしれないので連絡はしたほうがいい可能性はありますが...)(どんな重複起きてるか知らないのでテキトー発言だったらごめん、)

同一人物が複数回登録するのを防止。 なら...

[CaffeeLake]

SlackとDiscordで両方登録している人がいるけど、もう会費の支払いが終わっていて、消すに消せなかったりする。

[SnakeSneakS]

なっなるほど......

・slackアカウント消してdiscordアカウントだけでいいですかって連絡する(メアド同じならkibelaのアカウント変わらないだろうし)
・discordアカウントへ移行できるように「複数回登録可能」っていう選択肢もありかもしれない？移行したい人は新しくアカウント新規登録してもらうということで(その場合もしかしたら会計の人がポータルぽちぽちするのややこしくなるかもしれないけど)

の少なくとも二方針ありそうな感じですか...

[CaffeeLake]

メアドが異なるので、Kibelaとかのアカウントも異なるんですよね。
アカウントを複数持たれると、会費とかも含め色々と不都合が生じる。

[SnakeSneakS]

なるほど...
それじゃやっぱり、今重複してる人にdm送るなりして重複アカウント削除して、その後UNIQUE制約かけてアカウント必ず1つにするって方針が良さそうって感じですか〜(?)

[CaffeeLake]

手に負えなくなってきたので、強制UNIQUEをかけます。
どうせForntendにログの内容が流れないので、エラーメッセージの実装とかは飛ばします。

[CaffeeLake]

とりあえずDB側の変更はした。
17d5648