Skip to content

Conversation

@jaworek
Copy link

@jaworek jaworek commented Jul 20, 2022

Fresh NextJS project does not build after #3861 was merged. It looks like this is due to missing "require" path in "exports".

I'm not 100% sure why it is needed, but this suggestion from @satya164 seems to resolve the issue.

Repro: https://stackblitz.com/edit/nextjs-odbs3c

image

@vercel
Copy link

vercel bot commented Jul 20, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
query ❌ Failed (Inspect) Jul 20, 2022 at 4:46PM (UTC)

@codesandbox-ci
Copy link

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 2f89820:

Sandbox Source
@tanstack/query-example-react-basic Configuration
@tanstack/query-example-react-basic-typescript Configuration

@TkDodo
Copy link
Collaborator

TkDodo commented Jul 20, 2022

Thanks 🙏
I'm a noob in all this bundling things and it clearly shows 🙈

@TkDodo
Copy link
Collaborator

TkDodo commented Jul 20, 2022

Looks like this PR is a more complete fix?

@TkDodo
Copy link
Collaborator

TkDodo commented Jul 20, 2022

I'll merge the other PR. Thank you 🙏

@TkDodo TkDodo closed this Jul 20, 2022
@jaworek
Copy link
Author

jaworek commented Jul 20, 2022

Awesome! I agree that the other PR is better.

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