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 pt translations #31

Merged
merged 2 commits into from
Oct 19, 2022
Merged

added pt translations #31

merged 2 commits into from
Oct 19, 2022

Conversation

AndreGM
Copy link
Contributor

@AndreGM AndreGM commented Oct 9, 2022

What are you changing/introducing

Added pt translations (added pt folder)

What is the reason for changing/introducing

Added pt translations for portuguese users

QA

Q A
Is bugfix? no
New feature? no
Breaks BC? no
Tests pass?
Fixed issues comma-separated list of tickets # fixed by the PR, if any

'form_model_email_intro_hint' => 'O texto deste campo será usado como texto de introdução para o e-mail. Todas as variáveis ​​especificadas podem ser inseridas com {{attribute}}.',
'form_model_email_recipients_hint' => 'Se nenhum destinatário for fornecido, o envio do formulário não acionará um e-mail.',
'form_model_subject_hint' => 'Assunto do e-mail. Se não for especificado, o título do formulário será usado.',
'As Hidden Input Value' => '',
Copy link
Contributor

Choose a reason for hiding this comment

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

There are many missing translations. Maybe you where taking a look at the en file, but as those strings are english, they don't need translation. Here in the german file you can see they are all translated.

https://github.com/luyadev/luya-module-forms/blob/master/src/messages/de/forms.php

Copy link
Contributor Author

Choose a reason for hiding this comment

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

working on it 👍

@nadar
Copy link
Contributor

nadar commented Oct 12, 2022

Thanks for taking time, some missing translations, afterwards LGTM 👍

@AndreGM AndreGM requested a review from nadar October 18, 2022 00:52
@nadar nadar merged commit daebd54 into luyadev:master Oct 19, 2022
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