Skip to content
This repository has been archived by the owner on Apr 5, 2022. It is now read-only.

Add language pt #60

Closed
wants to merge 9 commits into from
Closed

Add language pt #60

wants to merge 9 commits into from

Conversation

multiarts
Copy link

Brazilian portuguese

@MrJuliuss MrJuliuss mentioned this pull request Jan 13, 2014
@MrJuliuss
Copy link
Owner

Hi, you put your files in src/lang/br/ but your pull request mentionned pt_br language. You need to differientate pt and pt_br for php language code : http://www.inmotionhosting.com/support/website/php/language-code-list

Please submit a new pull request (or modify it)

Julien

@GrahamCampbell
Copy link
Contributor

Also, squashing this into one commit would be good.

@multiarts
Copy link
Author

ok, updated

@MrJuliuss
Copy link
Owner

then the translations are for br or pt-br ? Please send a single commit in the request or send the pull request on the 1.1 branch (it's more convenient for me to merge in other branch)

@MrJuliuss
Copy link
Owner

Any news ?

@multiarts
Copy link
Author

http://www.inmotionhosting.com/support/website/php/language-code-list
pt-br Brasil, br is Bretan. Laravel but as a structure with short name then use br

@MrJuliuss
Copy link
Owner

Hi, larave use short language code, but pt-br is a short language code...
For Portuguese, you need to use pt or pt-br not br.

@multiarts
Copy link
Author

ok

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants