add schema public.attendees #178
Conversation
✅ Deploy Preview for vuefes-2024 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
無くて良いかと思います |
supabase/schema.sql
Outdated
| email varchar(100) not null unique, | ||
| avatar_url varchar(500) not null, | ||
| provider varchar(20) not null, | ||
| display_name varchar(24) not null, |
There was a problem hiding this comment.
ASK
display_name に NOT NULL 制約が入っていますが、
ソーシャルログインして取得され得る名前が初期値として入る想定で OK でしょうか？
There was a problem hiding this comment.
@jiyuujin
初回ログイン時は確かに display_name に初期値を入れないと空の状態ですね。失念していました。
となるとavatar_url は初期画像が入るので良いとして、receipt_idも今のままだと問題ですかね。
調べてみたところPostgreSQLのUNIQUEはnullを無視するようなので、receipt_idからもNOT NULL 制約を削除した方が良さそうでしょうか。
There was a problem hiding this comment.
display_name に初期値を入れないと空の状態
Vue Fes 2023 の例ですが、ソーシャルログインしたら user?.user_metadata.preferred_username || user?.user_metadata.name || '' を設定いただければ、初期値は設定されます
（ただし、空白値を許容しているので、NOT NULL 削除は無難かも？
https://github.com/vuejs-jp/vuefes-2023/blob/main/app/composables/useAuth.ts#L66
receipt_idも今のままだと問題ですかね。
結論からいうと今のまま NOT NULL 制約を付けていただいて問題ありません
その理由は public.attendees の更新のタイミングにあります。ソーシャルログインしただけですと auth.users という別テーブル（Supbase プロジェクトを作成して、最初から存在する内部テーブル）が更新されるだけにとどまります
では、public.attendees の更新のタイミングはいつ生まれるのか。それは、ネームカード追加（あるいは、編集）画面で更新する際に、public.attendees の更新のタイミングがあるので、その際は receipt_id に NOT NULL 制約を付けておけば必ず何かしら入力しないと通らなくなります
こんな感じの説明で理解 OK ですかね？
There was a problem hiding this comment.
@jiyuujin
ありがとうございます。理解しました。
display_nameのNOT NULL制約を削除したコミットをpushしました。
|
テーブル作成 |
Issue
https://github.com/vuejs-jp/vuefes-2024-backside/issues/208
レビュー完了後Figmaのテーブルコメントクローズ、実装に合わせて修正しておきます。
https://www.figma.com/board/mRTYOtFjg3L8ozmjEEhHLh/Vue-Fes-Japan-2024-%E3%81%8A%E7%B5%B5%E6%8F%8F%E3%81%8D%E3%83%9C%E3%83%BC%E3%83%89?node-id=0-1&t=OesnkqcE9bmT3Rou-0
Details
Ask
role varchar(16),に関して、CHECK 制約やENUM 型で制限を設けるべきでしょうか？DB設計あまり詳細に制約をつけたことがなく、勝手が分からず。