Skip to content
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

docs: Updates Cloudinary Astro Integration Guides #9550

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

colbyfayock
Copy link
Contributor

Description (required)

This makes two changes regarding Cloudinary in the Astro docs.

  1. Updates the primary Cloudinary integration page to walk through how to use Cloudinary with the newer Astro Cloudinary SDK as well as providing additional options depending on the person's needs.

  2. Updates a section in the Images documentation with regards to generating Cloudinary URLs with the Node.js SDK. It offers the Astro SDK as an alternative solution.

Related issues & labels (optional)

  • Suggested label: Add New Content

…o, updates Images section referring to the Cloudinary Node.js SDK to also include the Astro SDK
Copy link

netlify bot commented Oct 3, 2024

Deploy Preview for astro-docs-2 ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 7f32251
🔍 Latest deploy log https://app.netlify.com/sites/astro-docs-2/deploys/66feb5ecc7601b00080cd4ec
😎 Deploy Preview https://deploy-preview-9550--astro-docs-2.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.

@@ -497,7 +497,7 @@ export default defineConfig({

Image CDNs work with all Astro image options. Use an image's full URL as the `src` attribute in the `<Image />` component, an `<img>` tag, or in Markdown notation. For image optimization with remote images, also [configure your authorized domains or URL patterns](#authorizing-remote-images).

Alternatively, if the CDN provides a Node.js SDK, you can use that in your project. For example, [Cloudinary’s SDK](https://cloudinary.com/documentation/node_integration) can generate an `<img>` tag with the appropriate `src` for you.
Alternatively, the CDN may provide it's own SDKs to more easily integrate in an Astro project. For example, Cloudinary supports an [Astro SDK](https://astro.cloudinary.dev/) which allows you to easily drop in images with their CldImage component or a [Node.js SDK](https://cloudinary.com/documentation/node_integration) that can generate URLs to use with an `<img>` tag in a Node.js environment.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this feels a little Cloudinary heavy relative to the section that it's being described in, but the original intent was Node.js focused and I wanted to offer the Astro SDK as an alternative option now that it's available

@astrobot-houston
Copy link
Contributor

astrobot-houston commented Oct 3, 2024

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale File Note
en guides/images.mdx Source changed, localizations will be marked as outdated.
en guides/media/cloudinary.mdx Source changed, localizations will be marked as outdated.
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

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