LaLeV/wacrm
1
0
Fork 0
mirror of https://github.com/Luchexx/wacrm.git synced 2026-11-02 09:19:51 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

fix(db): create profile.account_id before is_account_member helper

Browse Source 
`is_account_member` is a `LANGUAGE sql` function, so Postgres parses
and validates its body at CREATE time — references to `profiles.account_id`
and `profiles.account_role` must already exist in the schema. The
original ordering created the helper before the ALTER TABLE that
added those columns, causing Supabase to abort with

    ERROR: column p.account_id does not exist (SQLSTATE 42703)

on every fresh apply. Reordering: the profile column-add now runs
immediately after the `accounts` / `account_invitations` tables and
before the helper. No semantic change elsewhere.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This commit is contained in:
Arnas Donauskas 2026-05-31 13:06:14 +03:00
parent 2f0334b0de
commit 55d4fedab9
No known key found for this signature in database
1 changed files with 18 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
supabase/migrations/017_account_sharing.sql
View File

@ -106,6 +106,24 @@ CREATE INDEX IF NOT EXISTS idx_account_invitations_account_pending
ALTER TABLE account_invitations ENABLE ROW LEVEL SECURITY; ALTER TABLE account_invitations ENABLE ROW LEVEL SECURITY;
-- ============================================================
-- PROFILE EXTENSION
--
-- account_role lives on profiles (not a separate memberships table)
-- because the design is one-account-per-user; this keeps reads cheap
-- (one row, already loaded by the auth hook).
--
-- Added BEFORE the is_account_member helper below because LANGUAGE
-- sql functions resolve column references at CREATE time (unlike
-- plpgsql, which defers to call time).
-- ============================================================
ALTER TABLE profiles
ADD COLUMN IF NOT EXISTS account_id UUID REFERENCES accounts(id) ON DELETE CASCADE,
ADD COLUMN IF NOT EXISTS account_role account_role_enum;
CREATE INDEX IF NOT EXISTS idx_profiles_account_role
ON profiles(account_id, account_role);
-- ============================================================ -- ============================================================
-- MEMBERSHIP HELPER -- MEMBERSHIP HELPER
-- --
@ -148,20 +166,6 @@ $$;
ALTER FUNCTION is_account_member(UUID, account_role_enum) OWNER TO postgres; ALTER FUNCTION is_account_member(UUID, account_role_enum) OWNER TO postgres;
GRANT EXECUTE ON FUNCTION is_account_member(UUID, account_role_enum) TO authenticated, service_role; GRANT EXECUTE ON FUNCTION is_account_member(UUID, account_role_enum) TO authenticated, service_role;
-- ============================================================
-- PROFILE EXTENSION
--
-- account_role lives on profiles (not a separate memberships table)
-- because the design is one-account-per-user; this keeps reads cheap
-- (one row, already loaded by the auth hook).
-- ============================================================
ALTER TABLE profiles
ADD COLUMN IF NOT EXISTS account_id UUID REFERENCES accounts(id) ON DELETE CASCADE,
ADD COLUMN IF NOT EXISTS account_role account_role_enum;
CREATE INDEX IF NOT EXISTS idx_profiles_account_role
ON profiles(account_id, account_role);
-- ============================================================ -- ============================================================
-- ADD account_id TO EVERY PARENT TENANT TABLE -- ADD account_id TO EVERY PARENT TENANT TABLE
-- --