-
-
Notifications
You must be signed in to change notification settings - Fork 66
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
Setting default language to OpenFoodFactsLanguage.WORLD and fields to… #75
Setting default language to OpenFoodFactsLanguage.WORLD and fields to… #75
Conversation
… ProductField.ALL if user does not provide them Adding tests to cover changes. Updating tests.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thank you!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I approved this too fast. In fact the OFF API has no "world" language. There is only a "world" country.
We should either force the apps to specify a language, or use English as the default. (which is what the API assumes if no language is passed).
@stephanegigandet , my bad. I will fix this and set it to EN. |
@stephanegigandet I have rebased my branch on the latest changes on the master. And with your changes on #76, no need anymore to set a default value for the language. I am only setting the field to ALL in case this was not provided on the configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
Setting default language to OpenFoodFactsLanguage.WORLD and fields to ProductField.ALL if user does not provide them.
Adding tests to cover changes.
Updating tests.