Skip to content

docs: update Image CDN docs for more correct patterns #732

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
Dec 12, 2023

Conversation

pieh
Copy link
Contributor

@pieh pieh commented Dec 11, 2023

Summary

Remote image patterns are regexes, and not URL patterns. While previous ones did actually work (as in allowed needed urls) - they were also somewhat too permissive. This change makes patterns less permissive while still correct

Test plan

  1. Visit the Deploy Preview (insert link to specific page) ...

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

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.

Copy link

netlify bot commented Dec 11, 2023

Deploy Preview for netlify-plugin-gatsby-demo-v5 ready!

Name Link
🔨 Latest commit e58ac62
🔍 Latest deploy log https://app.netlify.com/sites/netlify-plugin-gatsby-demo-v5/deploys/6576f4d97c98e30008bd2aec
😎 Deploy Preview https://deploy-preview-732--netlify-plugin-gatsby-demo-v5.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.

Copy link

netlify bot commented Dec 11, 2023

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

Name Link
🔨 Latest commit e58ac62
🔍 Latest deploy log https://app.netlify.com/sites/netlify-plugin-gatsby-demo/deploys/6576f4d93dd560000744340a
😎 Deploy Preview https://deploy-preview-732--netlify-plugin-gatsby-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.

@pieh pieh force-pushed the docs/image-cdn-remote-path-updates branch from 52f584c to e58ac62 Compare December 11, 2023 11:39
@pieh pieh changed the title chore: update Image CDN docs for more correct patterns docs: update Image CDN docs for more correct patterns Dec 11, 2023
@pieh pieh marked this pull request as ready for review December 11, 2023 14:46
@pieh pieh requested a review from a team as a code owner December 11, 2023 14:46
@pieh pieh merged commit ba5bc3d into main Dec 12, 2023
@pieh pieh deleted the docs/image-cdn-remote-path-updates branch December 12, 2023 07:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants