From 63a72eee42a2f4033a7135aa0fae00accba86c29 Mon Sep 17 00:00:00 2001
From: Tsering Paljor <tsering@baserow.io>
Date: Thu, 7 May 2026 14:00:31 +0700
Subject: [PATCH] [chore] Improve error message specificity and show Slack icon
 in Node History (#5313)

* Improve frontend error message specificity.

* Show the Slack icon in  Node History.

* Scope error handling to 400 status code

* Add test
---
 ...he_slack_icon_to_node_history_entries.json |  9 ++++++++
 ...end_error_message_to_be_more_specific.json |  9 ++++++++
 .../workflow/sidePanels/NodeHistory.vue       |  6 ++++--
 .../modules/core/plugins/clientHandler.js     | 14 +++++++++++++
 .../test/unit/core/utils/errors.spec.js       | 21 +++++++++++++++++++
 5 files changed, 57 insertions(+), 2 deletions(-)
 create mode 100644 changelog/entries/unreleased/refactor/added_the_slack_icon_to_node_history_entries.json
 create mode 100644 changelog/entries/unreleased/refactor/improved_the_frontend_error_message_to_be_more_specific.json

diff --git a/changelog/entries/unreleased/refactor/added_the_slack_icon_to_node_history_entries.json b/changelog/entries/unreleased/refactor/added_the_slack_icon_to_node_history_entries.json
new file mode 100644
index 0000000000..60e40f062a
--- /dev/null
+++ b/changelog/entries/unreleased/refactor/added_the_slack_icon_to_node_history_entries.json
@@ -0,0 +1,9 @@
+{
+    "type": "refactor",
+    "message": "Added the Slack icon to Node History entries.",
+    "issue_origin": "github",
+    "issue_number": null,
+    "domain": "automation",
+    "bullet_points": [],
+    "created_at": "2026-05-06"
+}
\ No newline at end of file
diff --git a/changelog/entries/unreleased/refactor/improved_the_frontend_error_message_to_be_more_specific.json b/changelog/entries/unreleased/refactor/improved_the_frontend_error_message_to_be_more_specific.json
new file mode 100644
index 0000000000..80cfbf632f
--- /dev/null
+++ b/changelog/entries/unreleased/refactor/improved_the_frontend_error_message_to_be_more_specific.json
@@ -0,0 +1,9 @@
+{
+    "type": "refactor",
+    "message": "Improved the frontend error message to be more specific.",
+    "issue_origin": "github",
+    "issue_number": null,
+    "domain": "core",
+    "bullet_points": [],
+    "created_at": "2026-05-06"
+}
\ No newline at end of file
diff --git a/web-frontend/modules/automation/components/workflow/sidePanels/NodeHistory.vue b/web-frontend/modules/automation/components/workflow/sidePanels/NodeHistory.vue
index 6f0ed76cec..82c0b65cc9 100644
--- a/web-frontend/modules/automation/components/workflow/sidePanels/NodeHistory.vue
+++ b/web-frontend/modules/automation/components/workflow/sidePanels/NodeHistory.vue
@@ -7,7 +7,8 @@
       <template #header="{ expanded }">
         <div class="node-history__header-row">
           <div class="node-history__header-icon">
-            <i :class="nodeIconClass"></i>
+            <i v-if="nodeIconClass" :class="nodeIconClass"></i>
+            <img v-else :alt="nodeType.name" :src="nodeType.image" />
           </div>
           <div class="node-history__header-info">
             <div>
@@ -100,7 +101,8 @@
 
     <div v-else class="node-history__header-row">
       <div class="node-history__header-icon">
-        <i :class="nodeIconClass"></i>
+        <i v-if="nodeIconClass" :class="nodeIconClass"></i>
+        <img v-else :alt="nodeType.name" :src="nodeType.image" />
       </div>
       <div class="node-history__header-info">
         <div
diff --git a/web-frontend/modules/core/plugins/clientHandler.js b/web-frontend/modules/core/plugins/clientHandler.js
index 10a43421dd..503d147d3d 100644
--- a/web-frontend/modules/core/plugins/clientHandler.js
+++ b/web-frontend/modules/core/plugins/clientHandler.js
@@ -278,6 +278,20 @@ export class ErrorHandler {
       return this.errorMap[this.code]
     }
 
+    const status = this.response?.status || 500
+    if (
+      this.detail &&
+      typeof this.detail === 'string' &&
+      status >= 400 &&
+      status < 500 &&
+      this.code != 'ERROR_REQUEST_BODY_VALIDATION'
+    ) {
+      return new ResponseErrorMessage(
+        this.app.$i18n.t('clientHandler.notCompletedTitle'),
+        this.detail
+      )
+    }
+
     return this.genericDefaultError()
   }
 
diff --git a/web-frontend/test/unit/core/utils/errors.spec.js b/web-frontend/test/unit/core/utils/errors.spec.js
index acb327233a..0f9e9457d2 100644
--- a/web-frontend/test/unit/core/utils/errors.spec.js
+++ b/web-frontend/test/unit/core/utils/errors.spec.js
@@ -83,6 +83,27 @@ describe('test error handling', () => {
       }
     }
   )
+  test('a 400 response with a string detail and no matching error map entry shows the detail as the message', async () => {
+    try {
+      await errorInterceptorWithStubAppAndStore()({
+        response: {
+          data: {
+            error: 'noMatchingEntryInSpecificErrorMap',
+            detail: 'The `minute` value must be greater or equal to 2.',
+          },
+          status: 400,
+        },
+      })
+    } catch (error) {
+      const message = error.handler.getMessage('name', {
+        doesntMatch: new ResponseErrorMessage('title', 'message'),
+      })
+      expect(message.title).toBe('clientHandler.notCompletedTitle')
+      expect(message.message).toBe(
+        'The `minute` value must be greater or equal to 2.'
+      )
+    }
+  })
   test('an 429 response results in a too many requests error', async () => {
     try {
       await errorInterceptorWithStubAppAndStore()({