More precise error msg on destination delete (#1148)

* Show original exception message on destination delete
Uninett · Jan 24, 2025 · 7564dae · 7564dae
1 parent 24d751c
commit 7564dae
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/changelog.d/1147.fixed.md b/changelog.d/1147.fixed.md
@@ -0,0 +1 @@
+Show relevant error message on destination delete by passing the original exception message to the UI.
diff --git a/src/argus/htmx/destination/views.py b/src/argus/htmx/destination/views.py
@@ -34,8 +34,8 @@ def delete_htmx(request, pk: int) -> HttpResponse:
     try:
         medium = api_safely_get_medium_object(destination.media.slug)
         medium.raise_if_not_deletable(destination)
-    except NotificationMedium.NotDeletableError:
-        error_msg = "That destination cannot be deleted."
+    except NotificationMedium.NotDeletableError as e:
+        error_msg = " ".join(e.args)
     else:
         destination.delete()
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		Show relevant error message on destination delete by passing the original exception message to the UI.