Merge pull request #320 from contentstack/fix/DX-2389

allow number, string, object, boolean, and RegExp as valid query para…

Author: sunil-lakshman

CHANGELOG.md

@@ -1,5 +1,10 @@
 ## Change log
 
+### Version: 3.25.2
+#### Date:  April-02-2025
+##### Fix:
+ - allow number, string, object, boolean, and RegExp as valid query parameters in sync method
+
 ### Version: 3.25.1
 #### Date:  April-01-2025
 ##### Fix:

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "contentstack",
-  "version": "3.25.1",
+  "version": "3.25.2",
   "description": "Contentstack Javascript SDK",
   "homepage": "https://www.contentstack.com/",
   "author": {

package.json

@@ -603,9 +603,16 @@ export default class Stack {
 
         if (params) {
             for (const key in params) {
+                const value = params[key];
                 if (params.hasOwnProperty(key)) {
-                    if (typeof params[key] !== "string" && typeof params[key] !== "number") {
-                        throw new Error(`Invalid parameter value for key "${key}": must be a string or number.`);
+                    if (
+                        typeof value !== "string" &&
+                        typeof value !== "number" &&
+                        typeof value !== "boolean" &&
+                        !(value instanceof RegExp) &&
+                        (typeof value !== "object" || value === null)
+                    ) {
+                        throw new Error(`Invalid parameter value for key "${key}": must be a string, number, object, boolean, or RegExp.`);
                     }
                     this._query[key] = params[key];
                 }

src/core/stack.js

@@ -0,0 +1,37 @@
+import * as Contentstack from '../..';
+
+const stack = Contentstack.Stack({ api_key: 'api_key', delivery_token: 'delivery_token', environment: 'environment', fetchOptions: {
+    logHandler: () => {
+
+    }
+}});
+
+describe('Sync Test', () => {
+    test('Sync init test', done => {
+        const response = makeSync({"init": true})
+        expect(response).not.toEqual(undefined)
+        done();
+    });
+
+    test('Sync with startdate test', done => {
+        const response = makeSync({"init": true, "start_from": "2025-04-02"})
+        expect(response).not.toEqual(undefined)
+        done();
+    });
+
+    test('Sync with locale test', done => {
+        const response = makeSync({"init": true, "locale": "en-us"})
+        expect(response).not.toEqual(undefined)
+        done();
+    });
+
+    test('Sync with contentTypeUid test', done => {
+        const response = makeSync({"init": true, "content_type_uid": "ct_uid"})
+        expect(response).not.toEqual(undefined)
+        done();
+    });
+});
+
+function makeSync(params: any) {
+    return stack.sync(params)
+}