Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add schema public.attendees #178

Merged
merged 4 commits into from
Jul 5, 2024
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 14 additions & 0 deletions supabase/schema.sql
Original file line number Diff line number Diff line change
Expand Up @@ -74,6 +74,20 @@ create table if not exists public.staffs (
updated_at timestamp with time zone default timezone('utc' :: text, now()) not null
);

create table if not exists public.attendees (
id uuid not null primary key default uuid_generate_v4(),
user_id uuid not null references auth.users on delete cascade,
email varchar(100) not null unique,
avatar_url varchar(500) not null,
provider varchar(20) not null,
display_name varchar(24) not null,
Copy link
Collaborator

Choose a reason for hiding this comment

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

ASK

display_name に NOT NULL 制約が入っていますが、
ソーシャルログインして取得され得る名前が初期値として入る想定で OK でしょうか?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jiyuujin
初回ログイン時は確かに display_name に初期値を入れないと空の状態ですね。失念していました。

となるとavatar_url は初期画像が入るので良いとして、receipt_idも今のままだと問題ですかね。
調べてみたところPostgreSQLのUNIQUEはnullを無視するようなので、receipt_idからもNOT NULL 制約を削除した方が良さそうでしょうか。

Copy link
Collaborator

@jiyuujin jiyuujin Jul 4, 2024

Choose a reason for hiding this comment

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

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 ですかね?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jiyuujin
ありがとうございます。理解しました。
display_nameのNOT NULL制約を削除したコミットをpushしました。

role varchar(16),
receipt_id varchar(20) not null unique,
activated_at timestamp with time zone,
created_at timestamp with time zone default timezone('utc' :: text, now()) not null,
updated_at timestamp with time zone default timezone('utc' :: text, now()) not null
);

-- *** Function definitions ***
create
or replace function public.create_admin_user() returns trigger as $ $ begin -- If user_role is 'admin', insert data into admin_users table
Expand Down