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

Added Spanish Language support #20

Closed
wants to merge 2 commits into from
Closed

Added Spanish Language support #20

wants to merge 2 commits into from

Conversation

javarb
Copy link
Contributor

@javarb javarb commented Apr 4, 2018

This version add strings variables in two languages (English and Spanish) into all the layouts in order to change the application language depending on the language configured in the device. For the application data two databases were used, one for each language.

By using two databases a problem arises if a person start the application in a language and after switch the device to the another one. Therefore, this approach will always load the Spanish Language database version for avoid any data loss risk, also considering this application version was intended to Spanish speakers.

This behavior can be modified changing in app/src/main/res/values/strings.xml:

gem-es.db3
to
gem.db3

@pslh
Copy link
Member

pslh commented Apr 5, 2018

For now I think we can use the approach whereby we use gem.db3 in strings.xml. I will update issue #19 based on these changes.

I see that there are many files which have no content changes but which have changed permissions (from 644 to 755) - could I ask you to please remove execute permission from the files README.md, LICENSE.txt, all html files, etc. and to renew this pull request - this way we see only the files that have really changed. Thanks.

@javarb
Copy link
Contributor Author

javarb commented Apr 5, 2018

Hello, I am doing so.

Permissions has been fixed. I pushed this to the new branch you created, so I am sending another pull-request from the 'spanish' branch in the corposso account.

@pslh
Copy link
Member

pslh commented Apr 5, 2018

Closing this PR - replaced by PR #21

@pslh pslh closed this Apr 5, 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.

2 participants