about summary refs log tree commit diff
diff options
context:
space:
mode:
authorFrederick Muriuki Muriithi2026-01-08 13:39:52 -0600
committerFrederick Muriuki Muriithi2026-01-08 14:07:21 -0600
commit4d7d1e2b6111bd17c6db6a9a270018b6b66a49f9 (patch)
tree201b14adabc322210ea59813731d61671ca5ba3e
parent0a6edab9f6c9e5cecb73dcd5427ea604c3395dfc (diff)
downloadgn-uploader-4d7d1e2b6111bd17c6db6a9a270018b6b66a49f9.tar.gz
Bugfix: Fix flash, and redirect to "Job Summary" page.
-rw-r--r--uploader/background_jobs.py7
1 files changed, 3 insertions, 4 deletions
diff --git a/uploader/background_jobs.py b/uploader/background_jobs.py
index c07e2f1..15fea15 100644
--- a/uploader/background_jobs.py
+++ b/uploader/background_jobs.py
@@ -173,11 +173,10 @@ def delete_single(job_id: uuid.UUID):
         try:
             job = jobs.job(conn, job_id, fulldetails=True)
             status = job["metadata"]["status"]
-                flash("We cannot delete a running job.", "alert alert-error")
-                # redirect below might be wrong in some cases. Redirect
-                # appropriately.
-                return redirect(url_for("background-jobs.list_jobs"))
             if status not in ("completed", "error", "stopped"):
+                flash("We cannot delete a running job.", "alert alert-danger")
+                return redirect(url_for(
+                    "background-jobs.job_summary", job_id=job_id))
 
             if request.method == "GET":
                 return render_template("background-jobs/delete-job.html",