fix pendin event notification

adobecom · Dec 17, 2024 · 43475cf · 43475cf
1 parent d543279
commit 43475cf
Show file tree

Hide file tree

Showing 4 changed files with 30 additions and 27 deletions.
diff --git a/libs/deps/mas/commerce.js b/libs/deps/mas/commerce.js
diff --git a/libs/deps/mas/mas.js b/libs/deps/mas/mas.js
diff --git a/libs/features/mas/dist/mas.js b/libs/features/mas/dist/mas.js
diff --git a/libs/features/mas/src/mas-element.js b/libs/features/mas/src/mas-element.js
@@ -153,6 +153,8 @@ export class MasElement {
         this.state = STATE_FAILED;
         this.update();
         this.log?.error('Failed:', { element: this.wrapperElement, error });
+        // Allow calling code to perform sync updates of this element
+        // before notifying observers about state change
         setImmediate(() => this.notify());
         return true;
     }
@@ -166,7 +168,8 @@ export class MasElement {
         if (options) this.options = options;
         this.state = STATE_PENDING;
         this.update();
-        setImmediate(() => this.notify());
+        // immediately notify observers about state change
+        this.notify();
         return this.version;
     }