darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-01-03 16:00:44 -05:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #349 from alphagov/fix-migration

Browse Source 
Remove the status from the group by clause of the update statement.
This commit is contained in:
Rebecca Law
2016-05-25 10:14:34 +01:00
parent 6735a7103c a0b974f4ac
commit 31f4bfd4f3
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")