-
Notifications
You must be signed in to change notification settings - Fork 787
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
feat(translation): add pt translation #440
Conversation
@Jpfonseca It's just Portuguese, right? If so, then leave just And please read the instruction for adding a new translation. Thanks in advance! |
@lex111 there are 2 types of portuguese : pt-pt and pt-br. That's why I kept the name of the file |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed, we already have Brazilian Portuguese so it only makes sense to specify the Portuguese variation :)
Thank you, this looks great to me. Feel free to do another pass through @lex111 and if it looks good let's merge this baby in
.all-contributorsrc
Outdated
@@ -381,6 +381,13 @@ | |||
"code", | |||
"bug" | |||
] | |||
}, | |||
{ | |||
"login": "Jpfonseca", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉
@Jpfonseca I understood, but the fact is that we change the original name of the locale, making it compatible with moment library, so we need the name of the locale to match the name of the locale in moment: Therefore, please change the code with P.S. @housseindjirdeh how correctly to describe this in the instruction? |
@lex111 Sorry that i didn't do that before . |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remained to do the last small edit.
@@ -20,6 +20,11 @@ export default [ | |||
name: 'Türkçe', | |||
}, | |||
{ | |||
code: 'pt-pt', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pt
instead pt-pt
.
Cheers thank you @lex111 and sorry I didn't realize that earlier. Added that step to our translations instructions. |
@lex111 I think you can merge it now .. |
@Jpfonseca No trouble at all! If anything, we need to apologize for not being super clear in the docs and having to tell you to change it up slghtly Thank you for this, really means a lot to take the time to get all the translations in ❤️ |
Added pt-pt as asked in #439 .