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

Added PL translations PO & MO #497

Merged
merged 1 commit into from
Jan 29, 2025
Merged

Conversation

jakubmieszczanin
Copy link
Contributor

Hi @andris9

I have translated the messages into Polish.

@CLAassistant
Copy link

CLAassistant commented Jan 29, 2025

CLA assistant check
All committers have signed the CLA.

@postalsys postalsys deleted a comment from CLAassistant Jan 29, 2025
@andris9
Copy link
Collaborator

andris9 commented Jan 29, 2025

Thanks. In order to accept the PR you would need to accept the CLA agreement shown above.

I also added machine-translated translations for input validation messages. Maybe you would want to check if these seem OK: https://github.com/postalsys/joi-messages/blob/master/translations/pl.po - most of these messages are never shown but some are needed, for example hostname related etc (for example, if you try to use a non standard hostname for the IMAP hostname field when filling the hosted authentication form)

@jakubmieszczanin
Copy link
Contributor Author

I signed the CLA and made a PR in the repository with translations of input validation messages - postalsys/joi-messages#5

@andris9
Copy link
Collaborator

andris9 commented Jan 29, 2025

Thanks! And I'll use your translations for joi-messages instead of the machine translated versions.

@andris9 andris9 merged commit b0b29e5 into postalsys:master Jan 29, 2025
3 checks passed
@andris9
Copy link
Collaborator

andris9 commented Jan 29, 2025

Seems to work
Screenshot 2025-01-29 at 10 20 00

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.

3 participants