From dcb45c64e8b434a8231bd06deac44edc9b440943 Mon Sep 17 00:00:00 2001 From: richard-dds Date: Tue, 7 Aug 2018 21:26:05 -0400 Subject: [PATCH] Alter old migrations so that they can downgrade The Role -> User foreign key constraint was preventing roles from being deleted once there were existing users referencing the role. I realized it was best to just pass on the downgrade and allow the tables to be deleted. --- .../versions/4ea5917e7781_add_default_atat_role.py | 4 +--- .../96a9f3537996_add_roles_and_permissions.py | 13 +------------ 2 files changed, 2 insertions(+), 15 deletions(-) diff --git a/alembic/versions/4ea5917e7781_add_default_atat_role.py b/alembic/versions/4ea5917e7781_add_default_atat_role.py index 78b6ef55..21b03166 100644 --- a/alembic/versions/4ea5917e7781_add_default_atat_role.py +++ b/alembic/versions/4ea5917e7781_add_default_atat_role.py @@ -34,6 +34,4 @@ def upgrade(): def downgrade(): - db = op.get_bind() - db.execute("DELETE FROM roles WHERE name = 'default'") - + pass diff --git a/alembic/versions/96a9f3537996_add_roles_and_permissions.py b/alembic/versions/96a9f3537996_add_roles_and_permissions.py index 4380208a..0729127e 100644 --- a/alembic/versions/96a9f3537996_add_roles_and_permissions.py +++ b/alembic/versions/96a9f3537996_add_roles_and_permissions.py @@ -169,15 +169,4 @@ def upgrade(): def downgrade(): - db = op.get_bind() - db.execute(""" - DELETE FROM roles - WHERE name IN ( - 'ccpo', - 'owner', - 'admin', - 'developer', - 'billing_auditor', - 'security_auditor' - ); - """) + pass