fix: handle alert returned async by parentAlertsAdd [LIBS-695] (#1388)

* fix: handle async parentAlertsAdd * chore: fix comment
dhis2 · Oct 9, 2024 · 9c989b2 · 9c989b2
1 parent 1713f0f
commit 9c989b2
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/services/alerts/src/useAlert.ts b/services/alerts/src/useAlert.ts
@@ -19,13 +19,18 @@ export const useAlert = (
                 typeof options === 'function' ? options(props) : options
 
             if (plugin && parentAlertsAdd && !showAlertsInPlugin) {
-                alertRef.current = parentAlertsAdd(
+                // Functions passed through post-robot are asynchronous
+                parentAlertsAdd(
                     {
                         message: resolvedMessage,
                         options: resolvedOptions,
                     },
                     alertRef
-                )
+                    // Conditional chaining gives backwards compatibility
+                    // with cli-app-scripts < 12
+                )?.then((newAlert: Alert) => {
+                    alertRef.current = newAlert
+                })
             } else {
                 alertRef.current = add(
                     {