fix: prevent frontend from crashing when api call fails

openstad · Feb 27, 2024 · e67605c · e67605c · github-actions · Feb 27, 2024
1 parent 8068c19
commit e67605c
Showing 1 changed file with 17 additions and 3 deletions.
diff --git a/packages/cms/lib/modules/resource-form-widgets/lib/api.js b/packages/cms/lib/modules/resource-form-widgets/lib/api.js
@@ -21,12 +21,19 @@ module.exports = async function(self, options) {
               await self.addOrUpdateNotification(item, 'User');
             } catch (error) {
               console.error(
-                'something went wrong when update admin confirmation settings to api',
+                'something went wrong when update user confirmation settings to api',
                 error.message
               );
             }
           } else {
-            await self.disableNotificationRuleSet(`User-${item.formName}`);
+            try {
+              await self.disableNotificationRuleSet(`User-${item.formName}`);
+            } catch (error) {
+              console.error(
+                'something went wrong when disabling user confirmation settings to api',
+                error.message
+              );
+            }
           }
 
           if (item.confirmationEnabledAdmin) {
@@ -39,7 +46,14 @@ module.exports = async function(self, options) {
               );
             }
           } else {
-            await self.disableNotificationRuleSet(`Admin-${item.formName}`);
+            try {
+              await self.disableNotificationRuleSet(`Admin-${item.formName}`);
+            } catch (error) {
+              console.error(
+                'something went wrong when disabling admin confirmation settings to api',
+                error.message
+              );
+            }
           }
         }
       })