move try/catch into main control flow and more from code review comments

server/notification-providers/heii-oncall.js

@@ -17,19 +17,24 @@ class HeiiOnCall extends NotificationProvider {
             payload["url"] = baseURL + getMonitorRelativeURL(monitorJSON.id);
         }
 
-        if (!heartbeatJSON) {
+        try {
-            // Testing or general notification like certificate expiry
+            if (!heartbeatJSON) {
-            payload["msg"] = msg;
+                // Testing or general notification like certificate expiry
-            return this.postNotification(notification, "alert", payload);
+                payload["msg"] = msg;
-        }
+                return this.postNotification(notification, "alert", payload);
+            }
 
-        if (heartbeatJSON.status === DOWN) {
+            if (heartbeatJSON.status === DOWN) {
-            return this.postNotification(notification, "alert", payload);
+                return this.postNotification(notification, "alert", payload);
-        }
+            }
 
-        if (heartbeatJSON.status === UP) {
+            if (heartbeatJSON.status === UP) {
-            return this.postNotification(notification, "resolve", payload);
+                return this.postNotification(notification, "resolve", payload);
+            }
+        } catch (error) {
+            this.throwGeneralAxiosError(error);
         }
+
     }
 
     /**
@@ -49,15 +54,11 @@ class HeiiOnCall extends NotificationProvider {
         };
 
         // Post to Heii On-Call Trigger https://heiioncall.com/docs#manual-triggers
-        try {
+        await axios.post(
-            await axios.post(
+            `https://heiioncall.com/triggers/${notification.heiiOnCallTriggerId}/${action}`,
-                `https://heiioncall.com/triggers/${notification.heiiOnCallTriggerId}/${action}`,
+            payload,
-                payload,
+            config
-                config
+        );
-            );
-        } catch (error) {
-            this.throwGeneralAxiosError(error);
-        }
 
         return "Sent Successfully";
     }

src/components/notifications/HeiiOnCall.vue

@@ -9,7 +9,7 @@
         ></HiddenInput>
     </div>
     <div class="mb-3">
-        <label for="heiioncall-trigger-id" class="form-label">{{ $t("Trigger ID") }}<span
+        <label for="heiioncall-trigger-id" class="form-label">Trigger ID<span
             style="color: red;"
         ><sup>*</sup></span></label>
         <HiddenInput
@@ -17,11 +17,9 @@
             autocomplete="false"
         ></HiddenInput>
     </div>
-    <div class="form-text">
+    <i18n-t tag="p" keypath="wayToGetHeiiOnCallDetails" class="form-text mt-3">
-        <i18n-t tag="p" keypath="wayToGetHeiiOnCallDetails" style="margin-top: 8px;">
+        <a href="https://heiioncall.com/docs" target="_blank">{{ $t("documentationOf", ["Heii On-Call"]) }}</a>
-            <a href="https://heiioncall.com/docs" target="_blank">{{ $t("documentationOf", ["Heii On-Call"]) }}</a>
+    </i18n-t>
-        </i18n-t>
-    </div>
 </template>
 
 <script>

src/lang/en.json

@@ -884,6 +884,6 @@
     "GrafanaOncallUrl": "Grafana Oncall URL",
     "Browser Screenshot": "Browser Screenshot",
     "What is a Remote Browser?": "What is a Remote Browser?",
-    "Trigger ID": "Trigger ID",
+    "wayToGetHeiiOnCallDetails": "How to get the Tigger ID and API Keys is explained in the {0}",
-    "wayToGetHeiiOnCallDetails": "How to get the Tigger ID and API Keys is explained in the {documentation}"
+    "documentationOf": "{0} Documentation"
 }