-
Notifications
You must be signed in to change notification settings - Fork 67
Cannot find module critters #157
Cannot find module critters #157
Comments
Hi @mannycolon 👋 This should have been fixed by netlify/zip-it-and-ship-it#295. Can you share the following parts of your build logs: If you have a public repo with a reproduction to share that would be awesome. |
Hi @erezrokah 👋 |
@erezrokah Here's the log
|
buildbot version: 2e994c874dd551982059545cd109ce3b21bce257 Here's a repo I forked to showcase the error when upgrading to Next v10.0.5 |
@lindsaylevine heya! continuing on from #103.
I'm not using the CLI, just the github continuous integration. here's the deploy link: https://app.netlify.com/teams/neef/builds/600d76a5b2138f000738f5dd |
I'm also having this error! Occurred when I updated |
Hi everyone, sorry for the delay to respond. @mannycolon I forked you repo and updated to For @neefrehman and @ezekielaquino can you share a public repo I can use to reproduce this? |
@erezrokah here you go: https://github.com/neefrehman/allergies.travel/tree/staging. Note that the bug is only on the staging branch, as main still uses |
@erezrokah It started working after your changes on netlify/build#2193, thanks |
Thanks @neefrehman that helps. I was able to reproduce the issue and will submit a fix soon. |
FYI, we haven't released the fix to our production env yet - I'll follow up here once it's released |
This fixed was rolled out to our production environment. Please comment if the issue still persists |
@erezrokah thanks for putting time on the fixes. I think there could be another use case missing as of today I found the error again.
|
@jhesgodi thanks for the repo and logs!! will defer to @erezrokah but just wanted to reopen and add that if you need features in the latest next version for whatever reason, another workaround is just to install critters. appreciate your patience!!! |
Thanks for reporting @jhesgodi - this is indeed a use case we didn't consider. We're discussing a long term solution so plugins can customize our bundling logic. |
See https://app.netlify.com/sites/silly-golick-1b4f9b/deploys/6009e6bf6ff93b00084d588f
netlify.toml
The text was updated successfully, but these errors were encountered: