Skip to content

Conversation

@PragalvaXFREZ
Copy link

@PragalvaXFREZ PragalvaXFREZ commented Feb 7, 2026

Description

  • Align isFullSiteBuild in gatsby-config.js with gatsby-node.js so both default to including all collections when BUILD_FULL_SITE is unset
  • Fix build log that always reported "includes all collections" due to [] == [] being false in JavaScript

Problem

gatsby-config.js used === "true" while gatsby-node.js used !== "false".
When BUILD_FULL_SITE is unset, the config excluded members/integrations from the file-system but gatsby-node still expected them, resulting in 637 pages silently missing with no error or placeholder.

Notes for Reviewers

Developers using make site or develop:lite are unaffected since those explicitly set BUILD_FULL_SITE=false and will continue to get fast builds with placeholders. This change only affects the "unset" case (e.g. make site-fast), which was already broken as pages silently 404'd with no placeholders shown.

Signed commits

  • Yes, I signed my commits.

Signed-off-by: Pragalva Sapkota <pragalva.sapkota@deerwalk.edu.np>
@l5io
Copy link
Contributor

l5io commented Feb 7, 2026

🚀 Preview for commit cfc7643 at: https://698794d6f32ec9dcaefbff4a--layer5.netlify.app

@saurabhraghuvanshii
Copy link
Member

@PragalvaXFREZ did you check on local by building, if commands are working properly.

@Bhumikagarggg
Copy link

@PragalvaXFREZ Thank you for your contribution! Let's discuss this during the website call tomorrow at 6:30 PM IST | 7 AM CST Add it as an agenda item to the meeting minutes, if you would 🙂

@PragalvaXFREZ
Copy link
Author

@PragalvaXFREZ did you check on local by building, if commands are working properly.

Hi @saurabhraghuvanshii, sorry for the late response. I did try running the build locally, but it hits the OS file watcher limit (ENOSPC) on my machine, which is unrelated to this change.

It would be helpful if you could verify the commands on your end to confirm the behavior.

@saurabhraghuvanshii
Copy link
Member

@PragalvaXFREZ can you share screenshot for error

@PragalvaXFREZ
Copy link
Author

@PragalvaXFREZ can you share screenshot for error

Please have a look @saurabhraghuvanshii
image

@saurabhraghuvanshii
Copy link
Member

@PragalvaXFREZ which command you're using for build

@PragalvaXFREZ
Copy link
Author

@PragalvaXFREZ which command you're using for build

I've used make build

@saurabhraghuvanshii
Copy link
Member

@PragalvaXFREZ try make site

@PragalvaXFREZ
Copy link
Author

@PragalvaXFREZ try make site

I have ran make site which works as expected, collections are excluded with 664 pages. Also managed to run make build and it shows 1291 pages with all collections included.

@l5io
Copy link
Contributor

l5io commented Feb 12, 2026

🚀 Preview for commit 36a4d6f at: https://698d9debb7ed84e529bd45e9--layer5.netlify.app

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants