Skip to content

fix: pass packagePath down to rsc edge manifest #2269

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

Merged
merged 2 commits into from
Aug 17, 2023

Conversation

lukasholzer
Copy link
Contributor

Description

Fixes an issue with monorepos and the new appDir where the packagePath was not passed down to the RSC edge manifest

I've deployed this change here on my test site with monorepo settings turned on, and the content-type is correctly set to text/x-component

curl -i -H "RSC: 1" https://lukas-next-runtime-demos-default.netlify.app/blog/erica/

CleanShot 2023-08-17 at 09 46 55

Documentation

Tests

You can test this change yourself like so:

  1. TODO

Relevant links (GitHub issues, etc.) or a picture of cute animal

@lukasholzer lukasholzer requested a review from a team as a code owner August 17, 2023 07:47
@netlify
Copy link

netlify bot commented Aug 17, 2023

Deploy Preview for netlify-plugin-nextjs-static-root-demo ready!

Name Link
🔨 Latest commit acfb5d4
🔍 Latest deploy log https://app.netlify.com/sites/netlify-plugin-nextjs-static-root-demo/deploys/64ddd246005b060008401ece
😎 Deploy Preview https://deploy-preview-2269--netlify-plugin-nextjs-static-root-demo.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link

netlify bot commented Aug 17, 2023

Deploy Preview for next-plugin-edge-middleware ready!

Name Link
🔨 Latest commit acfb5d4
🔍 Latest deploy log https://app.netlify.com/sites/next-plugin-edge-middleware/deploys/64ddd246e9bba80008d1a406
😎 Deploy Preview https://deploy-preview-2269--next-plugin-edge-middleware.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link

netlify bot commented Aug 17, 2023

Deploy Preview for netlify-plugin-nextjs-export-demo ready!

Name Link
🔨 Latest commit acfb5d4
🔍 Latest deploy log https://app.netlify.com/sites/netlify-plugin-nextjs-export-demo/deploys/64ddd2462ceb8700088d64ac
😎 Deploy Preview https://deploy-preview-2269--netlify-plugin-nextjs-export-demo.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@lukasholzer lukasholzer requested a review from pieh August 17, 2023 07:47
@lukasholzer lukasholzer self-assigned this Aug 17, 2023
@github-actions github-actions bot added the type: bug code to address defects in shipped code label Aug 17, 2023
@netlify
Copy link

netlify bot commented Aug 17, 2023

Deploy Preview for netlify-plugin-nextjs-demo ready!

Name Link
🔨 Latest commit acfb5d4
🔍 Latest deploy log https://app.netlify.com/sites/netlify-plugin-nextjs-demo/deploys/64ddd24678443900081d999c
😎 Deploy Preview https://deploy-preview-2269--netlify-plugin-nextjs-demo.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link

netlify bot commented Aug 17, 2023

Deploy Preview for netlify-plugin-nextjs-nx-monorepo-demo ready!

Name Link
🔨 Latest commit acfb5d4
🔍 Latest deploy log https://app.netlify.com/sites/netlify-plugin-nextjs-nx-monorepo-demo/deploys/64ddd2461d0e280008acdc64
😎 Deploy Preview https://deploy-preview-2269--netlify-plugin-nextjs-nx-monorepo-demo.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link

netlify bot commented Aug 17, 2023

Deploy Preview for netlify-plugin-nextjs-next-auth-demo ready!

Name Link
🔨 Latest commit acfb5d4
🔍 Latest deploy log https://app.netlify.com/sites/netlify-plugin-nextjs-next-auth-demo/deploys/64ddd246ed26bd000802a31d
😎 Deploy Preview https://deploy-preview-2269--netlify-plugin-nextjs-next-auth-demo.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link

netlify bot commented Aug 17, 2023

Deploy Preview for nextjs-plugin-custom-routes-demo ready!

Name Link
🔨 Latest commit acfb5d4
🔍 Latest deploy log https://app.netlify.com/sites/nextjs-plugin-custom-routes-demo/deploys/64ddd246f1621b00079419ac
😎 Deploy Preview https://deploy-preview-2269--nextjs-plugin-custom-routes-demo.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link

netlify bot commented Aug 17, 2023

Deploy Preview for next-i18next-demo ready!

Name Link
🔨 Latest commit acfb5d4
🔍 Latest deploy log https://app.netlify.com/sites/next-i18next-demo/deploys/64ddd246f1621b00079419a7
😎 Deploy Preview https://deploy-preview-2269--next-i18next-demo.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link

netlify bot commented Aug 17, 2023

Deploy Preview for next-plugin-canary ready!

Name Link
🔨 Latest commit acfb5d4
🔍 Latest deploy log https://app.netlify.com/sites/next-plugin-canary/deploys/64ddd2463e145b0008397c3a
😎 Deploy Preview https://deploy-preview-2269--next-plugin-canary.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@pieh pieh added the automerge label Aug 17, 2023
@kodiakhq kodiakhq bot merged commit 3517973 into main Aug 17, 2023
@kodiakhq kodiakhq bot deleted the fix/pass-package-path-to-rsc-edge-manifest branch August 17, 2023 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge type: bug code to address defects in shipped code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants