Skip to content

fixed locale pt-br not being used and added extra suport for pt-br - #1745

Merged
Corina (corinagum) merged 4 commits into
microsoft:masterfrom
pedropacheco92:master
Feb 20, 2019
Merged

Corina (corinagum) merged 4 commits into
microsoft:masterfrom
pedropacheco92:master

Conversation

@pedropacheco92

Copy link
Copy Markdown
Contributor

No description provided.

@msftclas

Microsoft Contribution License Agreements (msftclas) commented Feb 20, 2019 •

Copy link
Copy Markdown

CLA assistant check
All CLA requirements met.

@joaopedsa

Copy link
Copy Markdown

Nice Work!!

return 'pl-PL';
} else if (language.startsWith('pt')) {
if (language === 'pt-BR') {
if (language === 'pt-br') {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

could you explain why this is necessary?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

in the first line of this function, the language is setted to lower case, and this condidion was never true.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ah I understand now, thank you!

}

export default {
// FAILED_CONNECTION_NOTIFICATION: '',

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Are you able to fill the commented-out lines to finish up this localization file? :)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sure!

@corinagum Corina (corinagum) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I should have said this earlier, my apologies.

  • Please add your updates to the CHANGELOG.md
  • Please add your name to the LOCALIZATION.md

Thank you for your contribution!

@pedropacheco92

Copy link
Copy Markdown
Contributor Author

Fixed conflict

@corinagum

Copy link
Copy Markdown
Contributor

Will merge in once tests pass. Thanks again!

@corinagum
Corina (corinagum) merged commit db34ce4 into microsoft:master Feb 20, 2019
@corinagum

Copy link
Copy Markdown
Contributor

These changes will be in /latest/ of the 4.4 release. :)

@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.9%) to 54.268% when pulling ed5c6ab on pedropacheco92:master into c3b2a6d on Microsoft:master.

@coveralls

Coveralls (coveralls) commented Feb 21, 2019 •

Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.9%) to 54.268% when pulling c37e9a6 on pedropacheco92:master into c3b2a6d on Microsoft:master.

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.

5 participants