Skip to content

chore: remove netlify graph helper #2077

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 1 commit into from
May 1, 2023
Merged

chore: remove netlify graph helper #2077

merged 1 commit into from
May 1, 2023

Conversation

dwwoelfel
Copy link

Summary

Removes helper that make the Netlify Graph token available to the next runtime as it has reached its end of life.

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

https://github.com/netlify/pod-api-ecosystem/issues/174

Standard checks:

  • Check the Deploy Preview's Demo site for your PR's functionality
  • Add docs when necessary

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

@netlify
Copy link

netlify bot commented Apr 28, 2023

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

Name Link
🔨 Latest commit 8de1ac3
🔍 Latest deploy log https://app.netlify.com/sites/netlify-plugin-nextjs-nx-monorepo-demo/deploys/644c4a2788bafe000798cafe
😎 Deploy Preview https://deploy-preview-2077--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 Apr 28, 2023

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

Name Link
🔨 Latest commit 8de1ac3
🔍 Latest deploy log https://app.netlify.com/sites/netlify-plugin-nextjs-static-root-demo/deploys/644c4a279ac15800088c6a97
😎 Deploy Preview https://deploy-preview-2077--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 Apr 28, 2023

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

Name Link
🔨 Latest commit 8de1ac3
🔍 Latest deploy log https://app.netlify.com/sites/netlify-plugin-nextjs-export-demo/deploys/644c4a27ea8d1c000861494e
😎 Deploy Preview https://deploy-preview-2077--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.

@github-actions github-actions bot added the type: chore work needed to keep the product and development running smoothly label Apr 28, 2023
@netlify
Copy link

netlify bot commented Apr 28, 2023

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

Name Link
🔨 Latest commit 8de1ac3
🔍 Latest deploy log https://app.netlify.com/sites/netlify-plugin-nextjs-demo/deploys/644c4a287fdc200007a92230
😎 Deploy Preview https://deploy-preview-2077--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 Apr 28, 2023

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

Name Link
🔨 Latest commit 8de1ac3
🔍 Latest deploy log https://app.netlify.com/sites/nextjs-plugin-custom-routes-demo/deploys/644c4a2788bafe000798cb03
😎 Deploy Preview https://deploy-preview-2077--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.

@netlify
Copy link

netlify bot commented Apr 28, 2023

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

Name Link
🔨 Latest commit 8de1ac3
🔍 Latest deploy log https://app.netlify.com/sites/netlify-plugin-nextjs-next-auth-demo/deploys/644c4a27997d9c00084e82cc
😎 Deploy Preview https://deploy-preview-2077--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 Apr 28, 2023

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

Name Link
🔨 Latest commit 8de1ac3
🔍 Latest deploy log https://app.netlify.com/sites/next-plugin-edge-middleware/deploys/644c4a2708800e0008659425
😎 Deploy Preview https://deploy-preview-2077--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 Apr 28, 2023

Deploy Preview for next-plugin-canary ready!

Name Link
🔨 Latest commit 8de1ac3
🔍 Latest deploy log https://app.netlify.com/sites/next-plugin-canary/deploys/644c4a277959a000076899cd
😎 Deploy Preview https://deploy-preview-2077--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 Apr 28, 2023

Deploy Preview for next-i18next-demo ready!

Name Link
🔨 Latest commit 8de1ac3
🔍 Latest deploy log https://app.netlify.com/sites/next-i18next-demo/deploys/644c4a273cf7be0007ea2a2f
😎 Deploy Preview https://deploy-preview-2077--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.

@@ -116,14 +116,6 @@ const makeHandler = (conf: NextConfig, app, pageRoot, staticManifest: Array<[str
const query = new URLSearchParams(event.queryStringParameters).toString()
event.path = query ? `${event.path}?${query}` : event.path

const graphToken = event.netlifyGraphToken

Choose a reason for hiding this comment

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

Looks good to me @dwwoelfel. I believe this is the only place in the code we reference OneGraph.

@dwwoelfel dwwoelfel marked this pull request as ready for review May 1, 2023 17:07
@dwwoelfel dwwoelfel requested a review from a team May 1, 2023 17:07
Copy link

@nickytonline nickytonline left a comment

Choose a reason for hiding this comment

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

🚢

@dwwoelfel dwwoelfel merged commit b3275a1 into main May 1, 2023
@dwwoelfel dwwoelfel deleted the dww/nonegraph branch May 1, 2023 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: chore work needed to keep the product and development running smoothly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants