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

Portuguese translation to luya-bootstrap3 #28

Merged
merged 1 commit into from
Oct 22, 2019
Merged

Portuguese translation to luya-bootstrap3 #28

merged 1 commit into from
Oct 22, 2019

Conversation

wisnerjr
Copy link
Contributor

@wisnerjr wisnerjr commented Oct 2, 2019

What are you changing/introducing

Portuguese translation to luya-bootstrap3

What is the reason for changing/introducing

Fully update the portuguese translations.

QA

Q A
Is bugfix? no
New feature? no
Breaks BC? no
Tests pass? yes
Fixed issues luyadev/luya#1159

@codeclimate
Copy link

codeclimate bot commented Oct 2, 2019

Code Climate has analyzed commit 6b45eaf and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 95.3% (0.0% change).

View more on Code Climate.

@@ -107,7 +107,7 @@
'block_image_text_btnclass_label' => 'Classe CSS do Botão',

// 1.0.6
'block_map_api_key' => 'Maps API Key',
'block_map_api_key_help' => 'Since latest versions its recommend to provide a Google Maps API Key (https://developers.google.com/maps/documentation/embed/get-api-key)',
'block_map_api_key' => 'Mapemaneto Chaves API',

Choose a reason for hiding this comment

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

Suggested change
'block_map_api_key' => 'Mapemaneto Chaves API',
'block_map_api_key' => 'Chave da API do Maps',

@nadar nadar merged commit 7f565f1 into luyadev:master Oct 22, 2019
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