Remove hint about bad Meilisearch version

meilisearch · Jan 31, 2025 · e1ed70f · e1ed70f
1 parent 792ce67
commit e1ed70f
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 49 deletions.
diff --git a/src/errors/index.ts b/src/errors/index.ts
@@ -2,4 +2,3 @@ export * from "./meilisearch-api-error.js";
 export * from "./meilisearch-request-error.js";
 export * from "./meilisearch-error.js";
 export * from "./meilisearch-timeout-error.js";
-export * from "./version-hint-message.js";
diff --git a/src/errors/version-hint-message.ts b/src/errors/version-hint-message.ts
diff --git a/src/indexes.ts b/src/indexes.ts
@@ -5,12 +5,7 @@
  * Copyright: 2019, MeiliSearch
  */
 
-import {
-  MeiliSearchError,
-  MeiliSearchRequestError,
-  versionErrorHintMessage,
-  MeiliSearchApiError,
-} from "./errors/index.js";
+import { MeiliSearchError } from "./errors/index.js";
 import type {
   Config,
   SearchResponse,
@@ -373,22 +368,13 @@ class Index<T extends Record<string, any> = Record<string, any>> {
 
     // In case `filter` is provided, use `POST /documents/fetch`
     if (parameters.filter !== undefined) {
-      try {
-        const url = `indexes/${this.uid}/documents/fetch`;
-
-        return await this.httpRequest.post<
-          DocumentsQuery,
-          Promise<ResourceResults<D[]>>
-        >(url, parameters);
-      } catch (e) {
-        if (e instanceof MeiliSearchRequestError) {
-          e.message = versionErrorHintMessage(e.message, "getDocuments");
-        } else if (e instanceof MeiliSearchApiError) {
-          e.message = versionErrorHintMessage(e.message, "getDocuments");
-        }
-
-        throw e;
-      }
+      const url = `indexes/${this.uid}/documents/fetch`;
+
+      return await this.httpRequest.post<
+        DocumentsQuery,
+        Promise<ResourceResults<D[]>>
+      >(url, parameters);
+
       // Else use `GET /documents` method
     } else {
       const url = `indexes/${this.uid}/documents`;
@@ -601,19 +587,9 @@ class Index<T extends Record<string, any> = Record<string, any>> {
       : "documents/delete-batch";
     const url = `indexes/${this.uid}/${endpoint}`;
 
-    try {
-      const task = await this.httpRequest.post(url, params);
+    const task = await this.httpRequest.post(url, params);
 
-      return new EnqueuedTask(task);
-    } catch (e) {
-      if (e instanceof MeiliSearchRequestError && isDocumentsDeletionQuery) {
-        e.message = versionErrorHintMessage(e.message, "deleteDocuments");
-      } else if (e instanceof MeiliSearchApiError) {
-        e.message = versionErrorHintMessage(e.message, "deleteDocuments");
-      }
-
-      throw e;
-    }
+    return new EnqueuedTask(task);
   }
 
   /**

diff --git a/tests/documents.test.ts b/tests/documents.test.ts
@@ -172,9 +172,7 @@ describe("Documents tests", () => {
             "getDocuments should have raised an error when the route does not exist",
           );
         } catch (e: any) {
-          expect(e.message).toEqual(
-            "404: Not Found\nHint: It might not be working because maybe you're not up to date with the Meilisearch version that getDocuments call requires.",
-          );
+          expect(e.message).toEqual("404: Not Found");
         }
       });
 
@@ -191,8 +189,7 @@ describe("Documents tests", () => {
         } catch (e: any) {
           expect(e.message).toEqual(
             `Attribute \`id\` is not filterable. This index does not have configured filterable attributes.
-1:3 id = 1
-Hint: It might not be working because maybe you're not up to date with the Meilisearch version that getDocuments call requires.`,
+1:3 id = 1`,
           );
         }
       });
@@ -663,9 +660,7 @@ Hint: It might not be working because maybe you're not up to date with the Meili
             "deleteDocuments should have raised an error when the parameters are wrong",
           );
         } catch (e: any) {
-          expect(e.message).toEqual(
-            "Sending an empty filter is forbidden.\nHint: It might not be working because maybe you're not up to date with the Meilisearch version that deleteDocuments call requires.",
-          );
+          expect(e.message).toEqual("Sending an empty filter is forbidden.");
         }
       });
 
@@ -680,9 +675,7 @@ Hint: It might not be working because maybe you're not up to date with the Meili
             "deleteDocuments should have raised an error when the route does not exist",
           );
         } catch (e: any) {
-          expect(e.message).toEqual(
-            "404: Not Found\nHint: It might not be working because maybe you're not up to date with the Meilisearch version that deleteDocuments call requires.",
-          );
+          expect(e.message).toEqual("404: Not Found");
         }
       });