better feedback on delete entity

Author: simonredfern

src/routes/(protected)/dynamic-entities/system/+page.svelte

@@ -62,13 +62,27 @@
       });
 
       if (!response.ok) {
-        throw new Error("Failed to delete entity");
+        let errorMessage = "Failed to delete entity";
+        try {
+          const errorData = await response.json();
+          errorMessage = errorData.error || errorData.message || errorMessage;
+          console.error("Delete error response:", errorData);
+        } catch (e) {
+          const text = await response.text();
+          console.error("Delete error text:", text);
+          errorMessage = `${errorMessage}: ${response.status} ${response.statusText}`;
+        }
+        throw new Error(errorMessage);
       }
 
       alert("System dynamic entity deleted successfully");
       window.location.reload();
     } catch (error) {
-      alert("Failed to delete system dynamic entity");
+      const fullMessage =
+        error instanceof Error
+          ? error.message
+          : "Failed to delete system dynamic entity";
+      alert(`Failed to delete system dynamic entity:\n\n${fullMessage}`);
       console.error("Delete error:", error);
     }
   }

src/routes/(protected)/dynamic-entities/system/[id]/crud/+page.svelte

@@ -650,6 +650,11 @@
         <table class="min-w-full divide-y divide-gray-200 dark:divide-gray-700">
           <thead class="bg-gray-50 dark:bg-gray-900">
             <tr>
+              <th
+                class="px-6 py-3 text-left text-xs font-medium uppercase tracking-wider text-gray-500 dark:text-gray-400"
+              >
+                ID
+              </th>
               {#each Object.keys(properties).slice(0, 4) as fieldName}
                 <th
                   class="px-6 py-3 text-left text-xs font-medium uppercase tracking-wider text-gray-500 dark:text-gray-400"
@@ -669,7 +674,13 @@
           >
             {#each filteredRecords as record, index}
               {@const recordData = getRecordData(record)}
+              {@const recordId = getRecordId(record)}
               <tr class="hover:bg-gray-50 dark:hover:bg-gray-700/50">
+                <td
+                  class="max-w-xs truncate px-6 py-4 text-sm font-mono text-gray-600 dark:text-gray-400"
+                >
+                  {recordId || "-"}
+                </td>
                 {#each Object.keys(properties).slice(0, 4) as fieldName}
                   <td
                     class="max-w-xs truncate px-6 py-4 text-sm text-gray-900 dark:text-gray-100"
@@ -1063,6 +1074,14 @@
 
       <div class="bg-white p-6 dark:bg-gray-800">
         <dl class="space-y-4">
+          <div>
+            <dt class="text-sm font-medium text-gray-500 dark:text-gray-400">
+              ID
+            </dt>
+            <dd class="mt-1 text-sm font-mono text-gray-600 dark:text-gray-400">
+              {getRecordId(selectedRecord) || "-"}
+            </dd>
+          </div>
           {#each Object.entries(properties) as [fieldName, fieldDef]}
             {@const recordData = getRecordData(selectedRecord)}
             <div>

src/routes/api/dynamic-entities/[id]/+server.ts

@@ -143,24 +143,37 @@ export const DELETE: RequestHandler = async ({ params, locals }) => {
     logger.info(`Deleting dynamic entity: ${id}`);
 
     const endpoint = `/obp/v4.0.0/management/system-dynamic-entities/${id}`;
+    logger.info(`Calling DELETE ${endpoint}`);
     const response = await obp_requests.delete(endpoint, accessToken);
 
     logger.info("Dynamic entity deleted successfully");
     return json(response);
   } catch (err) {
     logger.error("Error deleting dynamic entity:", err);
 
+    // Log full error details
+    if (err && typeof err === "object") {
+      logger.error("Error details:", JSON.stringify(err, null, 2));
+    }
+
     let errorMessage = "Failed to delete dynamic entity";
     let obpErrorCode = undefined;
+    let detailedError = undefined;
 
     if (err instanceof Error) {
       errorMessage = err.message;
       if ("obpErrorCode" in err) {
         obpErrorCode = (err as any).obpErrorCode;
       }
+      if ("response" in err) {
+        detailedError = (err as any).response;
+      }
     }
 
-    const errorResponse: any = { error: errorMessage };
+    const errorResponse: any = {
+      error: errorMessage,
+      details: detailedError,
+    };
     if (obpErrorCode) {
       errorResponse.obpErrorCode = obpErrorCode;
     }