-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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
ES: Translation for security #4932
Conversation
Ping @nodejs/nodejs-es , plz have a review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✨LGTM✨
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small suggestions
@AugustinMauroy I know you have the best intentions with your LGTM, but if you're not part of the Website WG or not a Spanish translator, avoid stamping LGTM's as it gives a wrong impression that this PR got approved 🙈 |
suggested by @vrunoa in 4932
Thanks for the suggestions @vrunoa! 🙌 |
Propagate changes from #4941
Propagate changes from #4951
Can we merge it? :) |
@UlisesGascon I think someone from @nodejs/website still needs to approve it ? or maybe @nodejs/translators ? |
mmm... not sure, first time I do a translation 🤔 |
This is my first PR to do a full page translation. I am not sure 100% for some terms translation to Spanish but I tried to be very literal and I keep some english terms that are hard to translate to proper Spanish.
Please review it @nodejs/nodejs-es 🙏
Main changes
locale/es/docs/guides/index.md
to matchlocale/en/docs/guides/index.md
contentContext