Skip to content

Conversation

angelmarfil
Copy link
Contributor

Description (required)

Update node.mdx in spanish language

Related issues & labels (optional)

  • Closes #
  • Suggested label: i18n

Copy link

netlify bot commented Jul 7, 2025

Deploy Preview for astro-docs-2 ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 162f394
🔍 Latest deploy log https://app.netlify.com/projects/astro-docs-2/deploys/68a5f6f6eb24bc0008084743
😎 Deploy Preview https://deploy-preview-11993--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 project configuration.

@github-actions github-actions bot added the i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help! label Jul 7, 2025
@astrobot-houston
Copy link
Contributor

astrobot-houston commented Jul 7, 2025

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

File Note
es/guides/integrations-guide/node.mdx Localization changed, will be marked as complete. 🔄️
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.

Copy link
Contributor

@bjohansebas bjohansebas left a comment

Choose a reason for hiding this comment

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

LGTM, just a small detail

Co-authored-by: Sebastian Beltran <bjohansebas@gmail.com>
Copy link
Member

@yanthomasdev yanthomasdev left a comment

Choose a reason for hiding this comment

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

Thanks @angelmarfil, could you add the changes from your configuration-reference PR here, so we can merge it as one without links failing? That should unblock your other PRs too! 🙌

Copy link
Contributor

@bjohansebas bjohansebas left a comment

Choose a reason for hiding this comment

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

After reviewing it again, we still need to bring in the changes from this commit and remove the link I left as a suggestion.

After your other PRs are merged, they can be added.

Co-authored-by: Sebastian Beltran <bjohansebas@gmail.com>
Copy link
Contributor

@bjohansebas bjohansebas left a comment

Choose a reason for hiding this comment

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

LGTM! gracias

@yanthomasdev
Copy link
Member

Looks like we still don't have the changes from 32c4407, we need them translated before merging!

@bjohansebas
Copy link
Contributor

Hey, yeah, you need to add that content. Can you please add it?

I could have sworn it was there when I reviewed it, but maybe I confused it with other content and thought it had been integrated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
i18n Anything to do with internationalization & translation efforts - ask @YanThomas for help!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants