Skip to content

New Spanish language added via upload #3

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

Merged
merged 2 commits into from
Aug 20, 2024

Conversation

antoromani
Copy link
Contributor

New Spanish language added, please review

New Spanish language added
Copy link
Member

@Vovan-VE Vovan-VE left a comment

Choose a reason for hiding this comment

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

It's my fail to not mention ICU syntax in README. Sorry for that.

In general, all the "keys" in select and plural must be untouched, and all of then must include other key, even if empty other{}, or formatting will raise an error. Check any existing translations for reference (same filename same line number), better non-familar language.

Hello, I made the corrections. I also reviewed 4 more files. Thank you for the indications
@Vovan-VE
Copy link
Member

@antoromani , Thanks for contribution! Sorry for delay, I will check it as soon as possible.

@Vovan-VE Vovan-VE merged commit 64557f0 into cubux-net:master Aug 20, 2024
3 checks passed
@Vovan-VE
Copy link
Member

@antoromani , language published. Thanks for contribution! Feel free to collect remarks, if you will find some, to open new PR later.

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

Successfully merging this pull request may close these issues.

2 participants