Skip to content

Conversation

Dinotaurent
Copy link
Contributor

Description (required)

This PR removes the explanation about the hybrid mode in the Spanish documentation.

That content was removed from the Astro framework but was still present in the Spanish docs, which could cause confusion for readers.

This update aligns the Spanish documentation with the current English version.

Changes made only in the Spanish documentation.


Related issues & labels (optional)

  • Suggested label: i18n, docs, bug

First-time contributor to Astro Docs?

Yes, first-time contributor! 😊
Astro Discord: @Dinotaurent

The explanation of the use of ‘hybrid’ has been removed because it was eliminated but was still retained in the Spanish documentation, causing confusion.
Copy link

netlify bot commented Oct 6, 2025

Deploy Preview for astro-docs-2 ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit e1724bd
🔍 Latest deploy log https://app.netlify.com/projects/astro-docs-2/deploys/68ea6f9e0136f10008aa1768
😎 Deploy Preview https://deploy-preview-12514--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 Oct 6, 2025
@astrobot-houston
Copy link
Contributor

astrobot-houston commented Oct 6, 2025

Lunaria Status Overview

🌑 This pull request will not trigger status changes.

Learn more

Lunaria automatically ignores changes on specific PRs by adding a ignored keyword in its title. Found: i18nIgnore.

You can change this by either removing the keyword above from the PR's title, or modifying the ignoreKeywords property in your Lunaria configuration file.

Tracked Files

Note

The notes below indicate what would happen if the pull request is merged when triggering status changes. Since a ignored keyword was found in the PR's title, the status changes indicated below won't be applied.

File Note
es/reference/configuration-reference.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.

@astrobot-houston
Copy link
Contributor

Hello! Thank you for opening your first PR to Astro’s Docs! 🎉

Here’s what will happen next:

  1. Our GitHub bots will run to check your changes.
    If they spot any broken links you will see some error messages on this PR.
    Don’t hesitate to ask any questions if you’re not sure what these mean!

  2. In a few minutes, you’ll be able to see a preview of your changes on Netlify 🥳.

  3. One or more of our maintainers will take a look and may ask you to make changes.
    We try to be responsive, but don’t worry if this takes a few days.

@yanthomasdev
Copy link
Member

Thanks @Dinotaurent, I believe there are more changes to this file that need to be applied first. Here is a full list of commits in the English file since the last update to the translation: https://github.com/withastro/docs/commits/main/src/content/docs/en/reference/configuration-reference.mdx?since=2024-12-03T11:19:23.000Z

Feel free to check on our i18n dashboard

@sarah11918
Copy link
Member

Thank you for fixing this quite outdated docs page, @Dinotaurent ! We really appreciate it!

As Yan pointed out, there are other things that could be updated too. 😅 Are you available to make this page totally up to date?? If not, (we know that might be a lot to ask you!), we can merge only your current change. But if we can get you to update all the changes, that would be amazing for us.

Please let us know what you prefer! Thanks!

@Dinotaurent
Copy link
Contributor Author

Hi, this is really confusing to me, as this is my first time "collaborating" on another project, and I'm still not entirely familiar with the technicalities and the correct way to work on these teams. And to make things even more complicated, English isn't my native language, and I only speak a little bit of it hahahah 🫠, so I really don't understand everything because I'm just getting started. Sorry, I'm a little afraid of damaging something. 😅

@sarah11918
Copy link
Member

@Dinotaurent No problem at all!! We are very happy to have the change you contributed, and we can let someone else make more updates. 💜 Thank you again, and welcome to Team Docs!

@sarah11918 sarah11918 changed the title Remove outdated ‘hybrid’ explanation from Spanish docs (configuration-reference.mdx) [i18nIgnore] Remove outdated ‘hybrid’ explanation from Spanish docs (configuration-reference.mdx) Oct 11, 2025
@sarah11918 sarah11918 added the Merge Queue Approved and ready to be merged (wait for feature release if also labelled M-O-R)! label Oct 11, 2025
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 @Dinotaurent, good luck in your contributing journey! 🙌

@yanthomasdev yanthomasdev merged commit 9538037 into withastro:main Oct 11, 2025
10 checks passed
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! Merge Queue Approved and ready to be merged (wait for feature release if also labelled M-O-R)!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants