Merge pull request #125 from basehub-ai/jb/strip-height-in-basehub-image

Jb/strip height in basehub image

packages/basehub/CHANGELOG.md

@@ -1,5 +1,11 @@
 # basehub
 
+## 7.5.29
+
+### Patch Changes
+
+- Strip out height in BaseHubImage, as next/image doesn't require it
+
 ## 7.5.28
 
 ### Patch Changes

packages/basehub/package.json

@@ -2,7 +2,7 @@
   "name": "basehub",
   "description": "A very fast Headless CMS.",
   "author": "JB <[email protected]>",
-  "version": "7.5.28",
+  "version": "7.5.29",
   "license": "MIT",
   "repository": "basehub-ai/basehub",
   "bugs": "https://github.com/basehub-ai/basehub/issues",

packages/basehub/src/next/image/primitive.tsx

@@ -35,7 +35,10 @@ Expected origin to be one of:
           !param.startsWith("width=") &&
           !param.startsWith("quality=") &&
           !param.startsWith("w=") &&
-          !param.startsWith("q=")
+          !param.startsWith("q=") &&
+          // also strip height because next.js doesn't need it
+          !param.startsWith("h=") &&
+          !param.startsWith("height=")
         );
       });
       let newParams = [...filteredParams, ...params].join(",");
@@ -58,6 +61,9 @@ Expected origin to be one of:
     if (url.searchParams.has("format") === false) {
       url.searchParams.set("format", "auto");
     }
+    // also strip height because next.js doesn't need it
+    url.searchParams.delete("height");
+    url.searchParams.delete("h");
   }
 
   // pass old origin to new origin

playground/CHANGELOG.md

@@ -1,5 +1,12 @@
 # playground
 
+## 0.0.171
+
+### Patch Changes
+
+- Updated dependencies
+  - [email protected]
+
 ## 0.0.170
 
 ### Patch Changes

playground/package.json

@@ -1,7 +1,7 @@
 {
   "name": "playground",
   "private": true,
-  "version": "0.0.170",
+  "version": "0.0.171",
   "scripts": {
     "dev": "basehub dev & next dev --port 3003",
     "build": "basehub && next build",