Merge branch 'main' into feat/esm-configuration

CHANGELOG.md

@@ -1,3 +1,12 @@
+# [2.14.0](https://github.com/algolia/search-insights.js/compare/v2.13.0...v2.14.0) (2024-05-22)
+
+
+### Features
+
+* fast fail when sending no events ([#532](https://github.com/algolia/search-insights.js/issues/532)) ([de26c37](https://github.com/algolia/search-insights.js/commit/de26c37732d59cf2a2ec78768fed005e64dfc4c9))
+
+
+
 # [2.13.0](https://github.com/algolia/search-insights.js/compare/v2.12.0...v2.13.0) (2023-12-11)
 
 

lib/__tests__/_sendEvent.test.ts

@@ -737,6 +737,14 @@ describe("sendEvents", () => {
         }
       );
     });
+
+    it("should fail fast when sending no events via sendEvents", async () => {
+      const result = analyticsInstance.sendEvents([]);
+
+      expect(result).toBeInstanceOf(Promise);
+      expect(await result).toBe(false);
+      expect(fakeRequestFn).toHaveBeenCalledTimes(0);
+    });
   });
 
   it("applies default credentials when no custom ones are provided", () => {

lib/_sendEvent.ts

@@ -41,6 +41,10 @@ export function makeSendEvents(requestFn: RequestFnType) {
       return payload;
     });
 
+    if (events.length === 0) {
+      return Promise.resolve(false);
+    }
+
     return sendRequest(
       requestFn,
       this._ua,

package.json

@@ -1,7 +1,7 @@
 {
   "name": "search-insights",
   "description": "Library for reporting click, conversion and view metrics using the Algolia Insights API",
-  "version": "2.13.0",
+  "version": "2.14.0",
   "jsdelivr": "dist/search-insights.min.js",
   "main": "index-node.cjs.js",
   "types": "index-node.cjs.d.ts",