Skip to content
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.js 13? #88

Open
karlhorky opened this issue Oct 26, 2022 · 2 comments
Open

Support Next.js 13? #88

karlhorky opened this issue Oct 26, 2022 · 2 comments

Comments

@karlhorky
Copy link

Hi there, thanks for next-images! 🙌

Now that Next.js 13 is officially released, would you consider supporting this new version?

(maybe also version 11 and 12 as mentioned in the issues below)

@karlhorky
Copy link
Author

Or is the recommendation now to switch to next/image as mentioned here:

@zargold
Copy link

zargold commented May 11, 2023

Currently next/image has bugs with:

  • dealing with CDN: next/image does not work with CDN vercel/next.js#33488 which makes the change less attractive to someone using CDN for image hosting.
  • would not support expo
  • would not support legacy react apps
    --
    Possibly changes needed to get next-images to work are not such a large lift (current code is like 20 lines) and probably not much is needed to make it work in later versions of next.
    Only problem I've encountered so far with next-images is with dynamicAssetPrefix (if I turn this off my previous set up does work). So I say we at least dig in to find out what exactly are the problems with next-images that make it not work in version 12/13 (by the way my next-images config works fine in v12.2.x but not 12.3.4) so I doubt that version 11 is actually not working unless for some edge case that I didn't try.

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

No branches or pull requests

2 participants