-
Notifications
You must be signed in to change notification settings - Fork 27k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[docs] Add precision about pageExtensions
#40016
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, I'm not sure we need to document this here as we already explain this under the pageExtensions
doc here https://nextjs.org/docs/api-reference/next.config.js/custom-page-extensions
@ijjk ah! I totally missed it. I added the middleware way after I touched I'm fine closing this, it's just that I figured, if it happened to me, might happen to others. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sounds good to add this note, updated to link to the existing pageExtensions
doc. Thanks!
Adding a precision because when you change the
pageExtensions
configuration innext.config.js
,middleware.ts
isn't recognized anymore.Took me some time to debug this one, not sure if it should be considered a bug or just lack of documentation.
I'll let you be the judge of that.
Documentation / Examples
pnpm lint