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

Get Czech localizations to 100% #2204

Merged
merged 3 commits into from
Oct 6, 2020
Merged

Get Czech localizations to 100% #2204

merged 3 commits into from
Oct 6, 2020

Conversation

dusekdan
Copy link
Contributor

@dusekdan dusekdan commented Oct 1, 2020

Hello!

It's Hacktoberfest time again and I would like to contribute to AuthMeReloaded with localization to my native language.

In this pull request, I have:

  • translated all the TODO resource strings
  • added help_cz localization file and translated all the resource strings contained
  • unified the voice, or tone, which is used in both localization files; before this update, there was a mix up of the second case singular and second place plural when addressing a user of the plugin. I unified it to use 2nd case singular.

As time is of the essence in regards to Hacktoberfest, it would be greatly appreciated if you could merge my PR as soon as possible.

Thanks and keep up the good work!

Copy link

@MrIvanPlays MrIvanPlays left a comment

Choose a reason for hiding this comment

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

would like someone speaking ur language to confirm there are no offensive words.
otherwise thank you for your contribution

@TheDevTec
Copy link

There is one grammatical mistake. In messages_cz.yml on line 133. email_needed: '&3Pro ověřené tvé identity potřebujeme, abys ke svému účtu přidal svůj email.' Correctly is: "..Pro ověření tvé identity..".

Everything else should be ok!

@dusekdan
Copy link
Contributor Author

dusekdan commented Oct 4, 2020

Thank you, for the review, @TheDevTec, I fixed it in the commit above. Do you think it's mergeable now @MrIvanPlays?

@MrIvanPlays
Copy link

If there are no additional comments, I'm ready to merge it

@MrIvanPlays MrIvanPlays merged commit 99baed7 into AuthMe:master Oct 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

4 participants