darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2025-12-24 01:11:38 -05:00
Code Issues Packages Projects Releases Wiki Activity

Remove the status from the group by clause of the update statement.

Browse Source 
This script has been applied to all environments except live, this can be pushed through to live without issue because of how alembic works.
This commit is contained in:
Rebecca Law
2016-05-25 09:13:32 +01:00
parent 6735a7103c
commit a0b974f4ac
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
migrations/versions/0021_add_delivered_failed_counts.py
View File

@@ -26,13 +26,13 @@ def upgrade():
if x.job_id:
op.execute("update jobs set notifications_delivered = ("
"select count(status) from notifications where status = 'delivered' and job_id = '{}' "
"group by status, job_id)"
"group by job_id)"
"where jobs.id = '{}'".format(x.job_id, x.job_id))
op.execute("update jobs set notifications_failed = ("
"select count(status) from notifications "
"where status in ('failed','technical-failure', 'temporary-failure', 'permanent-failure') "
"and job_id = '{}' group by status, job_id)"
"and job_id = '{}' group by job_id)"
"where jobs.id = '{}'".format(x.job_id, x.job_id))
op.execute("update jobs set notifications_delivered = 0 where notifications_delivered is null")
op.execute("update jobs set notifications_failed = 0 where notifications_failed is null")