-
-
Notifications
You must be signed in to change notification settings - Fork 791
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
Support Next 13 #3979
Comments
@bitabs Ya you're probably right. Will mark this as ready to work on. |
@dillonraphael Sorry - do we have an ETA on this? It's a blocker for us unfortunately 😞 |
@bitabs const config = withBlitz({...stuff})
delete config.target
module.exports = config it should work |
That's not a bad shout @datner 😅. Works for the time being 🙏🏻 |
Thanks for the suggestion! I tried this and I'm still getting the same error as before, unfortunately. I also tried it in a fresh blitz app and got the same outcome. Do you have an example blitz app of this patch working? |
@blakeley either that or your js runtime is broken (which i doubt hahahah) |
If your running it via Vercel, be sure to invalidate your cache deployment. @blakeley |
Adding |
I ran it with The last few lines of my
And the vercel logs show the following:
which suggests that How could |
Current info on this…
Is there anything in this ticket that is not covered by the PR and ticket linke above? – I think it best to close this and maybe create separate tickets for open issues or add it to the todo list in #4070. |
@tordans what's missing for me personally is an overview about if we can finally upgrade to next 13. E.g. a documentation entry for migrating Note that upgrading to next 13 doesn't imply everything must work. For example, I'm not concerned about the optional/experimental app folder personally. I'd just like to have some sort of official statement/docs entry that upgrading to next 13 is safe and/or recommended even if there's some small caveats like "don't use app folder for now" |
app folder is a very cool feature though. Hopefully Blitz can support it soon |
closed in #4079 |
did you ever figure this out? still running into the same issue @blakeley |
Same here, can't deploy Next 13 app to Vercel (Both pages dir and app dir doesn't works). Log from vercel.com below:
|
I'm still having this problem after moving to the latest version of blitz
Vercel is still giving me the same error
Here's my next.config.js, and the build fails with and without the
|
Your're not deleting the target though, you're doing it too early 😅 // @ts-check
const { withBlitz } = require("@blitzjs/next")
const config = withBlitz({}) // withBlitz adds the 'target'
delete config.target
module.exports = config Not sure if it'll even help, but at least you'll delete the |
In case someone is still having this issue even after updating to the latest blitz/next versions, for me the issue was that I was still using the |
What do you want and why?
Please support Next 13 as Vercel deployment fails due to the following error:
The "target" property is no longer supported in next.config.js.
. I believe thewithBlitz({...})
maps config from Next 12 which hastarget
that is deprecated in Next 13.Possible implementation(s)
Bump Next 12 to 13
Additional context
The text was updated successfully, but these errors were encountered: