From 71cd319c51889f5bbd9a4303d9ba1bbb911d81f5 Mon Sep 17 00:00:00 2001 From: Hy Lac Date: Tue, 3 Feb 2026 20:58:14 -0500 Subject: [PATCH 1/2] Fix the migration to not include admin_info auth_id --- .../migrations/versions/ba76119b3e4c_update_user_system.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/backend/python/migrations/versions/ba76119b3e4c_update_user_system.py b/backend/python/migrations/versions/ba76119b3e4c_update_user_system.py index c6aca96..1a87dac 100644 --- a/backend/python/migrations/versions/ba76119b3e4c_update_user_system.py +++ b/backend/python/migrations/versions/ba76119b3e4c_update_user_system.py @@ -66,7 +66,7 @@ def upgrade() -> None: # Get existing admin records that need migration if 'admin_email' in admin_info_columns and 'admin_name' in admin_info_columns: existing_admins = connection.execute(sa.text(""" - SELECT admin_id, admin_name, admin_email, auth_id + SELECT admin_id, admin_name, admin_email FROM admin_info """)).fetchall() @@ -81,7 +81,7 @@ def upgrade() -> None: 'user_id': user_id, 'name': admin[1], 'email': admin[2], - 'auth_id': admin[3] if admin[3] else f'admin_{user_id}' + 'auth_id': f'admin_{user_id}' # THIS DOESN'T WORK LOL, THIS DOES NOT CORRESPOND TO THE FIREBASE AUTH_ID }) # Add user_id column as nullable first From 64590d9cf845497008caa072f74f77b649ebeb18 Mon Sep 17 00:00:00 2001 From: ludavidca Date: Tue, 3 Feb 2026 21:18:00 -0500 Subject: [PATCH 2/2] Removed Comment --- .../migrations/versions/ba76119b3e4c_update_user_system.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/backend/python/migrations/versions/ba76119b3e4c_update_user_system.py b/backend/python/migrations/versions/ba76119b3e4c_update_user_system.py index 1a87dac..6a0d836 100644 --- a/backend/python/migrations/versions/ba76119b3e4c_update_user_system.py +++ b/backend/python/migrations/versions/ba76119b3e4c_update_user_system.py @@ -81,7 +81,7 @@ def upgrade() -> None: 'user_id': user_id, 'name': admin[1], 'email': admin[2], - 'auth_id': f'admin_{user_id}' # THIS DOESN'T WORK LOL, THIS DOES NOT CORRESPOND TO THE FIREBASE AUTH_ID + 'auth_id': f'admin_{user_id}' }) # Add user_id column as nullable first