@advertol/core - npm Package Compare versions

Comparing version 1.0.3 to 1.0.4

CHANGELOG.md

@@ -5,2 +5,8 @@ # Changelog
 
+## [1.0.4][] - 2024-02-09
+
+### Changed
+
+- Update check when control should be triggered for specific zone
+
 ## [1.0.3][] - 2024-02-08
@@ -31,3 +37,4 @@
 
-[Unreleased]: https://github.com/niksy/advertol-core/compare/v1.0.3...HEAD
+[Unreleased]: https://github.com/niksy/advertol-core/compare/v1.0.4...HEAD
+[1.0.4]: https://github.com/niksy/advertol-core/compare/v1.0.3...v1.0.4
 [1.0.3]: https://github.com/niksy/advertol-core/compare/v1.0.2...v1.0.3
@@ -34,0 +41,0 @@ [1.0.2]: https://github.com/niksy/advertol-core/compare/v1.0.1...v1.0.2

index.cjs.js

@@ -417,12 +417,13 @@ 'use strict';
           isEmpty = _ref2.isEmpty;
-        /* eslint-disable no-undefined */
-        var triggerPromise = Promise.all([Promise.resolve(shouldTriggerControlCache[id] || false), Promise.resolve(onInitialControlTriggerCache[id] || undefined)]).then(function (_ref3) {
-          var isControlTriggered = _ref3[0],
-            initialControlTriggerResult = _ref3[1];
+        var triggerPromise = Promise.resolve(shouldTriggerControlCache[id] || false).then(function (isControlTriggered) {
           if (isControlTriggered) {
-            control.destroy({
-              isEmpty: isEmpty,
-              element: element,
-              id: id,
-              initialControlTriggerResult: initialControlTriggerResult
+            /* eslint-disable-next-line no-undefined */
+            Promise.resolve(onInitialControlTriggerCache[id] || undefined).then(function (initialControlTriggerResult) {
+              control.destroy({
+                isEmpty: isEmpty,
+                element: element,
+                id: id,
+                initialControlTriggerResult: initialControlTriggerResult
+              });
+              return;
             });
@@ -432,4 +433,2 @@ }
         });
-        /* eslint-enable */
-
         triggerPromises.push(triggerPromise);
@@ -436,0 +435,0 @@ });

index.esm.js

@@ -413,12 +413,13 @@ function _extends() {
           isEmpty = _ref2.isEmpty;
-        /* eslint-disable no-undefined */
-        var triggerPromise = Promise.all([Promise.resolve(shouldTriggerControlCache[id] || false), Promise.resolve(onInitialControlTriggerCache[id] || undefined)]).then(function (_ref3) {
-          var isControlTriggered = _ref3[0],
-            initialControlTriggerResult = _ref3[1];
+        var triggerPromise = Promise.resolve(shouldTriggerControlCache[id] || false).then(function (isControlTriggered) {
           if (isControlTriggered) {
-            control.destroy({
-              isEmpty: isEmpty,
-              element: element,
-              id: id,
-              initialControlTriggerResult: initialControlTriggerResult
+            /* eslint-disable-next-line no-undefined */
+            Promise.resolve(onInitialControlTriggerCache[id] || undefined).then(function (initialControlTriggerResult) {
+              control.destroy({
+                isEmpty: isEmpty,
+                element: element,
+                id: id,
+                initialControlTriggerResult: initialControlTriggerResult
+              });
+              return;
             });
@@ -428,4 +429,2 @@ }
         });
-        /* eslint-enable */
-
         triggerPromises.push(triggerPromise);
@@ -432,0 +431,0 @@ });

package.json

 {
   "name": "@advertol/core",
-  "version": "1.0.3",
+  "version": "1.0.4",
   "description": "Advertisement zones manager.",
@@ -5,0 +5,0 @@ "main": "index.cjs.js",

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

106250

increased by0.07%

No dependency changes