fix: migrate_roles (apache#16098)

opus-42 · Nov 14, 2021 · ee2cc8c · ee2cc8c
1 parent b1e6cd0
commit ee2cc8c
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/superset/migrations/versions/f6196627326f_update_chart_permissions.py b/superset/migrations/versions/f6196627326f_update_chart_permissions.py
@@ -38,6 +38,7 @@
 revision = "f6196627326f"
 down_revision = "143b6f2815da"
 
+NEW_PVMS = {"Chart": ("can_read",)}
 PVM_MAP = {
     Pvm("Chart", "can_get_data"): (Pvm("Chart", "can_read"),),
     Pvm("Chart", "can_post_data"): (Pvm("Chart", "can_read"),),
@@ -49,6 +50,7 @@ def upgrade():
     session = Session(bind=bind)
 
     # Add the new permissions on the migration itself
+    add_pvms(session, NEW_PVMS)
     migrate_roles(session, PVM_MAP)
     try:
         session.commit()