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

You can't switch language from settings anymore #3437

Closed
2 of 3 tasks
teolemon opened this issue Aug 19, 2020 · 11 comments · Fixed by #4192
Closed
2 of 3 tasks

You can't switch language from settings anymore #3437

teolemon opened this issue Aug 19, 2020 · 11 comments · Fixed by #4192
Labels
bug Produces incorrect or unexpected result or behaviour. i18n Translations release-blocking-issue
Milestone

Comments

@teolemon
Copy link
Member

teolemon commented Aug 19, 2020

You can't switch language from settings anymore

  • Taxonomy (works)
  • Robotoff (works)
  • UI (broken)
@teolemon teolemon added bug Produces incorrect or unexpected result or behaviour. i18n Translations release-blocking-issue labels Aug 19, 2020
@VaiTon
Copy link
Member

VaiTon commented Aug 20, 2020

Could you check with new commits?

@VaiTon VaiTon added this to the v3.4.2 milestone Aug 21, 2020
@teolemon
Copy link
Member Author

still here

@teolemon teolemon modified the milestones: v3.4.2, 4.0 Sep 5, 2020
@mrudultora
Copy link
Contributor

I even changed the language of my device (also from settings of openfoodfacts) , but then also the strings didn't changed. All the words were in english. Although, the data coming from server was in desired language.

@VaiTon
Copy link
Member

VaiTon commented Jan 12, 2021

There are a lot of incostincencies when using language. We use a lot Locale.getDefault() when we should be using the settings language.

@mrudultora
Copy link
Contributor

@VaiTon @teolemon I have pulled all the latest changes. When I install the app from android studio the language is changing. The one already installed from the playstore doesn't updates the language.
Had someone fixed this issue already ?
I am attaching the screenshots of both the cases.

img2

img1

In the second image the language is changed. Please, clarify this doubt.

@mrudultora
Copy link
Contributor

@teolemon @VaiTon Are we uploading app bundle on play console. If that's true, then this is the reason for language switching not working. Because, at the time of downloading play store will only download the language that is currently set in user's device. In this case, we have 2 options.
• Either we disable the split on the basis of language in the gradle file, but this will increase the app size because the app supports too many languages.
• Secondly, we can use Google Play Core Library to download resources at runtime, if needed. Have a look at this link.

@teolemon
Copy link
Member Author

teolemon commented Mar 6, 2021

It's not split related, since manually changing the language works fine

@mrudultora
Copy link
Contributor

Hi @teolemon , let me explain you completely. I downloaded the app from Playstore and at that time my device language was English. So, the strings were in English. Now, I manually changed my device language to Hindi from phone settings. Then I opened the app and the strings were still in English even after manually changing it. This clearly shows that strings.xml (hi) was not at all downloaded.
I guess your default device language is french. So, I suggest you to change your device language from phone settings (not from app settings) to any unknown language (change it to German or hindi). You will clearly see the difference. I am just putting my point because I recently faced the similar issue. The app bundle does not serve all the languages.

@mrudultora
Copy link
Contributor

Might be you are manually switching between french and English. Please try switching to other languages manually from phone settings and you will surely see the difference. I hope I am able to explain you my point :-)

@teolemon
Copy link
Member Author

teolemon commented Mar 7, 2021

I switched to Italian, and it works perfectly. French as well know.
I believe it might have been fixed in #3540
But I swear I could reproduce a couple of days ago 🤷

@mrudultora
Copy link
Contributor

mrudultora commented Mar 7, 2021

@teolemon So, I am unable to understand that why I am getting a contradicting result. Would you please look at this screen recording.

Screenrecorder-2021-03-07-22-06-14-627_486x1054.mp4

I changed my device language to french but the app was still in english (tried everything to get the french language in the app) . Video is a bit long as my phone was lagging.
I downloaded the latest beta version of the app from google playstore Also, the pull request related to #3540 is closed (not merged ). @sarah541 had fixed a NPE related issue on language change in #3556 . This is a different issue, as their is no npe here but then also language is not changing.
Sorry, for pinging you agaiin and again.

@teolemon teolemon linked a pull request Aug 27, 2021 that will close this issue
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Produces incorrect or unexpected result or behaviour. i18n Translations release-blocking-issue
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants