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

Feat: Add swahili language translation #953

Merged
merged 1 commit into from
Dec 3, 2018

Conversation

letelete
Copy link
Contributor

Fixes #Issue_Number

Please Add Screenshots If there are any UI changes.

2
3
1

Please make sure these boxes are checked before submitting your pull request - thanks!

  • Apply the AndroidStyle.xml style template to your code in Android Studio.

  • Run the unit tests with ./gradlew check to make sure you didn't break anything

  • If you have multiple commits please combine them into one commit by squashing them.

Copy link
Collaborator

@luckyman20 luckyman20 left a comment

Choose a reason for hiding this comment

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

@letelete I see some of the strings aren't translated, any specific reason for that.

@letelete
Copy link
Contributor Author

letelete commented Nov 25, 2018

I've removed strings which, imho, shouldn't be translated like app name and location. Am I wrong with this?

@luckyman20
Copy link
Collaborator

App name and location are fine but strings like Password, filter etc. should be translated.

@letelete
Copy link
Contributor Author

Oh, yeah, looks like I missed a couple of these. I'm going to translate them now.

@letelete letelete force-pushed the swahili-translation branch from 92dd00f to 4c7af6c Compare November 25, 2018 07:11
@letelete
Copy link
Contributor Author

Updated! Should be fine already.

app/src/main/res/values-sw/strings.xml Outdated Show resolved Hide resolved
app/src/main/res/values-sw/strings.xml Outdated Show resolved Hide resolved
@letelete letelete force-pushed the swahili-translation branch from 4c7af6c to 2fc05db Compare November 25, 2018 07:23
@letelete
Copy link
Contributor Author

Is there anything else I should fix?

@letelete letelete force-pushed the swahili-translation branch from 2fc05db to e3d8d39 Compare November 25, 2018 08:25
@luckyman20
Copy link
Collaborator

@letelete Travis is failing, fix that.

@letelete
Copy link
Contributor Author

letelete commented Nov 25, 2018

It's passing locally.

Travis logs: Error: Invalid --abi armeabi-v7a for the selected target. I'll try to recommit it because it's been passing earlier and I just changed the string.

image

@letelete letelete force-pushed the swahili-translation branch from e3d8d39 to 5f4bf12 Compare November 25, 2018 08:52
@miPlodder miPlodder merged commit 47c76a9 into openMF:development Dec 3, 2018
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