-
-
Notifications
You must be signed in to change notification settings - Fork 204
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
i18n/es: Spanish translation #818
Conversation
Thanks for the PR, much appreciated. If there's anything you need help with, please ask. |
Translation finished |
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.
Thank you very much!
The translations are looking good. There are a couple of minor issues left though. Unless you spot any further mistakes or potential improvements, this can be merged once the issues have been fixed.
Co-authored-by: Sebastian Meyer <mail@bastimeyer.de>
Co-authored-by: Sebastian Meyer <mail@bastimeyer.de>
Co-authored-by: Sebastian Meyer <mail@bastimeyer.de>
Thanks for your help! |
I going to squash the commits when merging anyway, so it's not a problem with the commit messages. Ideally, every translation namespace (each file) would have its own commit when a new language gets added, but rebasing the commits isn't that important now. The most important thing is that each change can be reviewed properly, and this was the case here. I'll merge this later this evening when I had the time to look over it one last time. |
No description provided.