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

Incorrect filter locale displayed after language change #2971

Closed
TheHasagi opened this issue Jul 30, 2019 · 9 comments
Closed

Incorrect filter locale displayed after language change #2971

TheHasagi opened this issue Jul 30, 2019 · 9 comments

Comments

@TheHasagi
Copy link
Contributor

Incorrect filter translation after language is changed (See the screenshot)

Preconditions are:

Switch device language for RU (example), AdGuard language is RU.

Steps to reproduce

  1. Switch language of AdGuard to EN
  2. Open AdGuard - Settings - DNS filtering - DNS requests blocking

Expected behavior

SDN filter should look with the selected locale (EN)

Actual behavior

SDN filter shown in RU translation

Screenshot:

img

@ameshkov
Copy link
Member

ameshkov commented Aug 1, 2019

Hmm, @admitrevskiy where does this translation come from? I thought there are no translations of the DNS filter lists?

@admitrevskiy
Copy link

We store SDN filter name in strings.xml. Looks like on the first SDN filter initialization selected locale was ru. Anyway, I think we should hide this phrase and make it not translatable.

@ameshkov
Copy link
Member

ameshkov commented Aug 1, 2019

Well, in the case of custom DNS filter lists, the user enters their names manually so there's no need to bother.

This one is preinstalled so ideally, it should be "translatable". It's just we'd better do it in runtime.

@lancelot-moon
Copy link

Yes, AdGuard SDN filter in DNS filter should be translated.
#2964
Thanks.

@lancelot-moon
Copy link

lancelot-moon commented Aug 24, 2019

This isn't fixed in v3.2.140.

My Android system language is Chinese.

After cleanly installing v3.2.140, if changing AdGuard's language from system (Chinese) to English or any other language, AG SDN filter still shows Chinese.

Afterwards, I tried to change Android system language from Chinese to English.
But AG SDN filter still show Chinese even if changing AdGuard's language to any other languages.

@TheHasagi
Copy link
Contributor Author

@lancelot-moon

As you can see the milestone is set to 3.3 and it cannot be fixed in 3.2.140. Please, stand by.

@lancelot-moon
Copy link

lancelot-moon commented Aug 24, 2019

@TheHasagi
I knew what you said.
Then you should not mark that you had fixed this in the changelog.
https://adguard.com/en/versions/android/release.html
[Fixed] "DNS filters" screen uses incorrect locale after language change #2971

@TheHasagi
Copy link
Contributor Author

Fair enough :)

@admitrevskiy
Copy link

Done.

@zzebrum zzebrum closed this as completed Oct 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants