fix(jodie-core): Allow 0 for homepagePageLimit and `homepageProje…

…ctLimit` (#1311) Co-authored-by: Lennart <lekoarts@gmail.com>
LekoArts · Feb 11, 2024 · 93f8009 · 93f8009
1 parent 3cc2d8c
commit 93f8009
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/.changeset/short-pans-guess.md b/.changeset/short-pans-guess.md
@@ -0,0 +1,5 @@
+---
+"@lekoarts/gatsby-theme-jodie-core": patch
+---
+
+Allow `0` for `homepagePageLimit` and `homepageProjectLimit` theme options. Previously that value would have no effect and the default value was used.
diff --git a/themes/gatsby-theme-jodie-core/utils/default-options.mjs b/themes/gatsby-theme-jodie-core/utils/default-options.mjs
@@ -6,8 +6,8 @@ export const withDefaults = (themeOptions) => {
   const pagesPath = themeOptions.pagesPath || `content/pages`
   const formatString = themeOptions.formatString || `DD.MM.YYYY`
   const navigation = themeOptions.navigation || []
-  const homepagePageLimit = themeOptions.homepagePageLimit || 9999
-  const homepageProjectLimit = themeOptions.homepageProjectLimit || 3
+  const homepagePageLimit = themeOptions.homepagePageLimit ?? 9999
+  const homepageProjectLimit = themeOptions.homepageProjectLimit ?? 3
   const mdx = typeof themeOptions.mdx === `undefined` ? true : themeOptions.mdx
   const sharp = typeof themeOptions.sharp === `undefined` ? true : themeOptions.sharp