Skip to content

Commit

Permalink
Add migration to set default alert selector
Browse files Browse the repository at this point in the history
In commits fc1e1f7 and e44fcdb a new Selector option was added to
alerts, which may be "first", "min" or "max".  This migration sets the
default to "first" for existing alerts.
  • Loading branch information
eradman committed Oct 18, 2024
1 parent 04a25f4 commit ebb8700
Showing 1 changed file with 27 additions and 0 deletions.
27 changes: 27 additions & 0 deletions migrations/versions/1655999df5e3_default_alert_selector.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
"""set default alert selector
Revision ID: 1655999df5e3
Revises: 7205816877ec
Create Date: 2024-10-18 11:24:49
"""

from redash.models import db

# revision identifiers, used by Alembic.
revision = '1655999df5e3'
down_revision = '7205816877ec'
branch_labels = None
depends_on = None


def upgrade():
db.session.execute("""
UPDATE alerts
SET options = jsonb_set(options, '{selector}', '"first"')
WHERE options->>'selector' IS NULL;
""")
db.session.commit()

def downgrade():
pass

0 comments on commit ebb8700

Please sign in to comment.