-
Notifications
You must be signed in to change notification settings - Fork 24
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
Language Option for Food Search #76
Comments
I just checked and it seems the App is either using german or "world" for the api. These are the supported countries: |
I was pretty sure I saw French (but I don't want to embarass myself too much so I'll check again). |
Added temporary fix to #76. It's missing a propper check if the country code is supported. Allowed calls on main thread for now to improve stability and reduce crashes. When the database call structure has been redone, this should be switched off again. Fixes #75, Fixes #65 (fixed with the key generation change)
|
There's unfortunately another case: The country specific search can find some product(s) but not all so that the world search shows more options. Edit: Wouldn't it be better to just use "world" until any manual exclusive country setting is there? This would be very easy to change just within /app/src/main/java/org/secuso/privacyfriendlyfoodtracker/network/ApiManager.java
Otherwise there are many products missing. |
Yes it would. I spoke about this on OFF slack channel a few months ago and got confirmation. I am surprised this is still an open issue, this makes the application unusable for those affected by the bug. |
When I search for food, I am greeted with options in French. I do not see any option to change the language, but I do see that the API you use does support English.
Did I miss something or is there no option to change languages from within the app?
The text was updated successfully, but these errors were encountered: