Skip to content

fix: avoid undefined isExport in slug routes#4044

Merged
tangly1024 merged 3 commits into
notionnext-org:mainfrom
qianzhu18:codex/fix-export-slug-routes
May 18, 2026
Merged

fix: avoid undefined isExport in slug routes#4044
tangly1024 merged 3 commits into
notionnext-org:mainfrom
qianzhu18:codex/fix-export-slug-routes

Conversation

@qianzhu18
Copy link
Copy Markdown
Contributor

Summary

  • replace direct isExport() calls in [prefix] slug route pages with a local static export flag
  • keep export-mode revalidate behavior unchanged for one-, two-, and multi-segment slug routes
  • prevent prerender failures when these routes build without an in-scope isExport helper

Verification

  • yarn type-check
  • ./node_modules/.bin/eslint pages/[prefix]/index.js pages/[prefix]/[slug]/index.js pages/[prefix]/[slug]/[...suffix].js (existing hook warnings only)
  • yarn export locally progressed through compile and page data collection without reproducing the earlier ReferenceError: isExport is not defined before being stopped during long Notion block prefetching

@netlify
Copy link
Copy Markdown

netlify Bot commented May 15, 2026

Deploy Preview for notionnext-netlify ready!

Name Link
🔨 Latest commit 8319ed8
🔍 Latest deploy log https://app.netlify.com/projects/notionnext-netlify/deploys/6a09e1946e698000085a7b5c
😎 Deploy Preview https://deploy-preview-4044--notionnext-netlify.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@tangly1024 tangly1024 merged commit 2a78fd4 into notionnext-org:main May 18, 2026
9 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants