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

Update languagePt-BR #271

Open
wants to merge 2 commits into
base: development
Choose a base branch
from

Conversation

menestrel-mkmb
Copy link

I was trying the app, and realize that the translation was for an old version, and it's not complete anymore, so I updated the language file.

For the language itself, a lot of text was splitted with "some" + "text", I was thinking about some linter, but on the "new" part I didn't see any of that, so I just got the rule of thumb: don't change unless test it, but...
I'm having an issue with signing keys, and after some gradle dep ('AudioTagger.updateMediaStore' and others), so I can't build the full project. I just PR this because you describe on the README:

Where for your new Language you have to add a new LanguageData(flag, name, languageCode) into the _supportedLanguages list, then, a new switch case in _loadLocale() function with your languageCode and return your new language File, open a Pull Request and after checking I will merge it!.

If you don't feel like doing this last step, you can still send me your new Language File via PullRequest and I will do it.

As you welcome just plain language files and do the configuration for yourself, I think that's OK to request this PR.

If you really want me to test it despise the README content, has been more than 2 years that I don't use Android Studio and code Flutter, so on my timeboxing for today I couldn't run the build to test, if someone could help me to realize how I can start from 'nothing installed' to AVD test, I can do that.

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.

1 participant