Skip to content

feat: skip run if @netlify/plugin-nextjs installed #1536

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 6 commits into from
Aug 16, 2022
Merged

Conversation

ericapisani
Copy link

@ericapisani ericapisani commented Aug 15, 2022

Disable in order to avoid conflicts should the old package be installed in the project.

Summary

Disables the runtime if it detects that the old plugin is installed in the project. Show a message prompting user to remove the package from their project so they can use the runtime instead.

Test plan

  • Confirm message shows in project that has @netlify/plugin-js installed (test done in demos/default)
  • Confirm site without @netlify/plugin-js works as expected

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

Issue

baby-tiger


🧪 Once merged, make sure to update the version if needed and that it was published correctly.

disable in order to avoid conflicts should the old package be installed in the project.
@ericapisani ericapisani added the type: feature code contributing to the implementation of a feature and/or user facing functionality label Aug 15, 2022
@ericapisani ericapisani self-assigned this Aug 15, 2022
@netlify
Copy link

netlify bot commented Aug 15, 2022

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

Name Link
🔨 Latest commit b9c2fd3
🔍 Latest deploy log https://app.netlify.com/sites/netlify-plugin-nextjs-demo/deploys/62fb9950160f7100097ba0f6
😎 Deploy Preview https://deploy-preview-1536--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 settings.

@netlify
Copy link

netlify bot commented Aug 15, 2022

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

Name Link
🔨 Latest commit b9c2fd3
🔍 Latest deploy log https://app.netlify.com/sites/netlify-plugin-nextjs-nx-monorepo-demo/deploys/62fb99506991020009b9d672
😎 Deploy Preview https://deploy-preview-1536--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 settings.

@netlify
Copy link

netlify bot commented Aug 15, 2022

Deploy Preview for next-hp-edge-demo ready!

Name Link
🔨 Latest commit b9c2fd3
🔍 Latest deploy log https://app.netlify.com/sites/next-hp-edge-demo/deploys/62fb9950a715220008b49474
😎 Deploy Preview https://deploy-preview-1536--next-hp-edge-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 settings.

@netlify
Copy link

netlify bot commented Aug 15, 2022

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

Name Link
🔨 Latest commit b9c2fd3
🔍 Latest deploy log https://app.netlify.com/sites/netlify-plugin-nextjs-export-demo/deploys/62fb99506bf4b9000937a70f
😎 Deploy Preview https://deploy-preview-1536--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 settings.

@netlify
Copy link

netlify bot commented Aug 15, 2022

Deploy Preview for next-plugin-rsc-demo ready!

Name Link
🔨 Latest commit b9c2fd3
🔍 Latest deploy log https://app.netlify.com/sites/next-plugin-rsc-demo/deploys/62fb9950cb9d2c0009825d73
😎 Deploy Preview https://deploy-preview-1536--next-plugin-rsc-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 settings.

@netlify
Copy link

netlify bot commented Aug 15, 2022

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

Name Link
🔨 Latest commit b9c2fd3
🔍 Latest deploy log https://app.netlify.com/sites/next-plugin-edge-middleware/deploys/62fb995085bdd70008c4ae48
😎 Deploy Preview https://deploy-preview-1536--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 settings.

@netlify
Copy link

netlify bot commented Aug 15, 2022

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

Name Link
🔨 Latest commit b9c2fd3
🔍 Latest deploy log https://app.netlify.com/sites/netlify-plugin-nextjs-next-auth-demo/deploys/62fb9950160f7100097ba0fb
😎 Deploy Preview https://deploy-preview-1536--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 settings.

@netlify
Copy link

netlify bot commented Aug 15, 2022

Deploy Preview for next-plugin-canary ready!

Name Link
🔨 Latest commit b9c2fd3
🔍 Latest deploy log https://app.netlify.com/sites/next-plugin-canary/deploys/62fb995073deeb0008d687c7
😎 Deploy Preview https://deploy-preview-1536--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 settings.

@netlify
Copy link

netlify bot commented Aug 15, 2022

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

Name Link
🔨 Latest commit b9c2fd3
🔍 Latest deploy log https://app.netlify.com/sites/netlify-plugin-nextjs-static-root-demo/deploys/62fb99508a21bb0009327ae0
😎 Deploy Preview https://deploy-preview-1536--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 settings.

@netlify
Copy link

netlify bot commented Aug 15, 2022

Deploy Preview for next-i18next-demo ready!

Name Link
🔨 Latest commit b9c2fd3
🔍 Latest deploy log https://app.netlify.com/sites/next-i18next-demo/deploys/62fb995024092b00088f43ac
😎 Deploy Preview https://deploy-preview-1536--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 settings.

@netlify
Copy link

netlify bot commented Aug 15, 2022

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

Name Link
🔨 Latest commit b9c2fd3
🔍 Latest deploy log https://app.netlify.com/sites/nextjs-plugin-custom-routes-demo/deploys/62fb9950578c18000977a364
😎 Deploy Preview https://deploy-preview-1536--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 settings.

@ericapisani ericapisani marked this pull request as ready for review August 15, 2022 19:11
@ericapisani ericapisani requested a review from a team August 15, 2022 19:11
Copy link

@sarahetter sarahetter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@kodiakhq kodiakhq bot merged commit 0b4cf8c into main Aug 16, 2022
@kodiakhq kodiakhq bot deleted the ep/add-detection branch August 16, 2022 13:33
kodiakhq bot pushed a commit that referenced this pull request Aug 17, 2022
…1548)

* feat: revert "feat: skip run if @netlify/plugin-nextjs installed (#1536)"

This reverts commit 0b4cf8c.

* feat: changing back package.json to plugin-nextjs

* chore: lint fix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge type: feature code contributing to the implementation of a feature and/or user facing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants