coryHawkvelt
adfeb1bd0f
Remove UniqueConstraint('id') from all create_table calls in the initial
migration (40 occurrences) and the bulk constraint additions from the
superadmin migration (43 create + 43 drop). These were redundant with
PrimaryKeyConstraint('id') which already guarantees uniqueness.
Also removes duplicate unique enforcement on superadmins.email and
superadmin_sessions.token (kept the unique indexes, dropped the
table-level UniqueConstraints).
Fixes the root cause in BaseModel by removing unique=True from the id
column definition, which was causing Alembic autogenerate to produce
these redundant constraints.
Renames idx_cert_audit_org to ix_certificate_audit_logs_organization_id
to follow Alembic naming conventions.
101 lines
5.2 KiB
Python
101 lines
5.2 KiB
Python
"""Superadmin
|
|
|
|
Revision ID: b4cd6c6b3b1c
|
|
Revises: 6a4c4ed4a5c6
|
|
Create Date: 2026-04-08 16:55:52.646980
|
|
|
|
"""
|
|
from alembic import op
|
|
import sqlalchemy as sa
|
|
|
|
|
|
# revision identifiers, used by Alembic.
|
|
revision = 'b4cd6c6b3b1c'
|
|
down_revision = '6a4c4ed4a5c6'
|
|
branch_labels = None
|
|
depends_on = None
|
|
|
|
|
|
def upgrade():
|
|
# --- Create superadmin tables (not captured by auto-generation) ---
|
|
op.create_table(
|
|
'superadmins',
|
|
sa.Column('id', sa.String(length=36), nullable=False),
|
|
sa.Column('email', sa.String(length=255), nullable=False),
|
|
sa.Column('password_hash', sa.String(length=255), nullable=False),
|
|
sa.Column('full_name', sa.String(length=255), nullable=True),
|
|
sa.Column('is_active', sa.Boolean(), nullable=False, server_default=sa.text('true')),
|
|
sa.Column('last_login_at', sa.DateTime(), nullable=True),
|
|
sa.Column('created_at', sa.DateTime(), nullable=False),
|
|
sa.Column('updated_at', sa.DateTime(), nullable=False),
|
|
sa.Column('deleted_at', sa.DateTime(), nullable=True),
|
|
sa.PrimaryKeyConstraint('id'),
|
|
)
|
|
op.create_index(op.f('ix_superadmins_email'), 'superadmins', ['email'], unique=True)
|
|
|
|
op.create_table(
|
|
'superadmin_sessions',
|
|
sa.Column('id', sa.String(length=36), nullable=False),
|
|
sa.Column('superadmin_id', sa.String(length=36), nullable=False),
|
|
sa.Column('token', sa.String(length=255), nullable=False),
|
|
sa.Column('expires_at', sa.DateTime(), nullable=False),
|
|
sa.Column('last_activity_at', sa.DateTime(), nullable=False),
|
|
sa.Column('ip_address', sa.String(length=45), nullable=True),
|
|
sa.Column('user_agent', sa.Text(), nullable=True),
|
|
sa.Column('revoked_at', sa.DateTime(), nullable=True),
|
|
sa.Column('revoked_reason', sa.String(length=255), nullable=True),
|
|
sa.Column('created_at', sa.DateTime(), nullable=False),
|
|
sa.Column('updated_at', sa.DateTime(), nullable=False),
|
|
sa.Column('deleted_at', sa.DateTime(), nullable=True),
|
|
sa.ForeignKeyConstraint(['superadmin_id'], ['superadmins.id']),
|
|
sa.PrimaryKeyConstraint('id'),
|
|
)
|
|
op.create_index(op.f('ix_superadmin_sessions_superadmin_id'), 'superadmin_sessions', ['superadmin_id'])
|
|
op.create_index(op.f('ix_superadmin_sessions_token'), 'superadmin_sessions', ['token'], unique=True)
|
|
|
|
op.create_table(
|
|
'superadmin_audit_logs',
|
|
sa.Column('id', sa.String(length=36), nullable=False),
|
|
sa.Column('superadmin_id', sa.String(length=36), nullable=False),
|
|
sa.Column('action', sa.String(length=100), nullable=False),
|
|
sa.Column('resource_type', sa.String(length=50), nullable=False),
|
|
sa.Column('resource_id', sa.String(length=36), nullable=True),
|
|
sa.Column('org_id', sa.String(length=36), nullable=True),
|
|
sa.Column('user_id', sa.String(length=36), nullable=True),
|
|
sa.Column('ip_address', sa.String(length=45), nullable=True),
|
|
sa.Column('user_agent', sa.Text(), nullable=True),
|
|
sa.Column('request_id', sa.String(length=100), nullable=True),
|
|
sa.Column('extra_data', sa.JSON(), nullable=True),
|
|
sa.Column('success', sa.Boolean(), nullable=False, server_default=sa.text('true')),
|
|
sa.Column('error_message', sa.String(length=500), nullable=True),
|
|
sa.Column('created_at', sa.DateTime(), nullable=False),
|
|
sa.Column('updated_at', sa.DateTime(), nullable=False),
|
|
sa.Column('deleted_at', sa.DateTime(), nullable=True),
|
|
sa.ForeignKeyConstraint(['superadmin_id'], ['superadmins.id']),
|
|
sa.PrimaryKeyConstraint('id'),
|
|
)
|
|
op.create_index(op.f('ix_superadmin_audit_logs_superadmin_id'), 'superadmin_audit_logs', ['superadmin_id'])
|
|
op.create_index(op.f('ix_superadmin_audit_logs_action'), 'superadmin_audit_logs', ['action'])
|
|
op.create_index(op.f('ix_superadmin_audit_logs_resource_type'), 'superadmin_audit_logs', ['resource_type'])
|
|
op.create_index(op.f('ix_superadmin_audit_logs_resource_id'), 'superadmin_audit_logs', ['resource_id'])
|
|
op.create_index(op.f('ix_superadmin_audit_logs_org_id'), 'superadmin_audit_logs', ['org_id'])
|
|
op.create_index(op.f('ix_superadmin_audit_logs_user_id'), 'superadmin_audit_logs', ['user_id'])
|
|
|
|
|
|
def downgrade():
|
|
# --- Drop superadmin tables (reverse order due to FK dependencies) ---
|
|
op.drop_index(op.f('ix_superadmin_audit_logs_user_id'), table_name='superadmin_audit_logs')
|
|
op.drop_index(op.f('ix_superadmin_audit_logs_org_id'), table_name='superadmin_audit_logs')
|
|
op.drop_index(op.f('ix_superadmin_audit_logs_resource_id'), table_name='superadmin_audit_logs')
|
|
op.drop_index(op.f('ix_superadmin_audit_logs_resource_type'), table_name='superadmin_audit_logs')
|
|
op.drop_index(op.f('ix_superadmin_audit_logs_action'), table_name='superadmin_audit_logs')
|
|
op.drop_index(op.f('ix_superadmin_audit_logs_superadmin_id'), table_name='superadmin_audit_logs')
|
|
op.drop_table('superadmin_audit_logs')
|
|
|
|
op.drop_index(op.f('ix_superadmin_sessions_token'), table_name='superadmin_sessions')
|
|
op.drop_index(op.f('ix_superadmin_sessions_superadmin_id'), table_name='superadmin_sessions')
|
|
op.drop_table('superadmin_sessions')
|
|
|
|
op.drop_index(op.f('ix_superadmins_email'), table_name='superadmins')
|
|
op.drop_table('superadmins')
|