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

translations: add a spanish translation #9

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

SrEstegosaurio
Copy link

An incomplete and probably partially erroneous Spanish translation of the webpage. I wanted to know if this was what I was supposed to do since I haven't messed with Astro in my whole life.

Also: @eerii feel free to nitpick as much as you want please! I don't expect this to be anything near finished yet. :)

@SrEstegosaurio SrEstegosaurio requested a review from eerii May 5, 2024 01:00
Copy link

@eerii eerii left a comment

Choose a reason for hiding this comment

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

Thank you so much for starting this translation! The base here is great, I just had a few nitpicks and some tiny corrections, but feel free to change anything you like. I'll also try to make some of the missing translations as well.

public/locales/es/translations.json Outdated Show resolved Hide resolved
public/locales/es/translations.json Outdated Show resolved Hide resolved
public/locales/es/translations.json Outdated Show resolved Hide resolved
public/locales/es/translations.json Outdated Show resolved Hide resolved
public/locales/es/translations.json Outdated Show resolved Hide resolved
@mrshmllow mrshmllow added the translation Translation of existing documentation label May 5, 2024
@SrEstegosaurio
Copy link
Author

Me reading the suggestions be like: dammmnn, how could I not have thought of that?

Thanks @eerii for the much needed corrections. I approved everything, we could get picky once we run out of things to translate.

mrshmllow

This comment was marked as outdated.

@SrEstegosaurio
Copy link
Author

Sorry for the delay but my final exams are near and I don't have much time atm. I would probably get this merged soon. (Hope that's today).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
translation Translation of existing documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants