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

Missing default translation string setting for updater.errors #185

Closed
akoepcke opened this issue Aug 14, 2022 · 2 comments
Closed

Missing default translation string setting for updater.errors #185

akoepcke opened this issue Aug 14, 2022 · 2 comments
Assignees
Labels
bug Something isn't working (correctly). fixed in upcoming release Fixed in the upcoming release!

Comments

@akoepcke
Copy link

Describe the bug
When having German as OS language setting, the translation strings for the updater.error modal are incomplete. The text does not fall back to default English, but displays the string names instead.

To Reproduce
Steps to reproduce the behavior:

  1. Switch Mac OS system language settings to German (I'm guessing any other non-english language works as well).
  2. Kill WLAN/network connection
  3. Klick "Check for updates..."

Expected behavior
When the modal appears that tells me it can't check for updates, I expect the modal to have a clearly understandable error message. Since the whole app is in english, I expect that error message to be in plain English as well.

Screenshots
Bildschirmfoto 2022-08-14 um 20 49 50

Required information

  • Did you consult the FAQ in the README? not applicable
  • Did you try "Fix My Valet"? not applicable
  • OS: macOS Monterey, including all system updates until August 14th, 2022
  • OS region setting: Germany
  • OS language setting: German(primary), English
  • PHP Monitor Version 5.4.1 (912)
@akoepcke akoepcke added the bug Something isn't working (correctly). label Aug 14, 2022
@nicoverbruggen
Copy link
Owner

nicoverbruggen commented Aug 14, 2022

Thanks for reporting, I will definitely look at this and ensure this is fixed in the next release! Do any other strings appear untranslated or is this the only one you noticed?

Edit: Ah, I changed the keys, so this message is broken on any system configuration. Whoops!

nicoverbruggen added a commit that referenced this issue Aug 14, 2022
This ensures that the "failed to check for update" modal now displays
the correct localized string.
@nicoverbruggen nicoverbruggen added the fixed in upcoming release Fixed in the upcoming release! label Aug 14, 2022
@akoepcke
Copy link
Author

Ah, nice. That was an easy fix then 😁
And no, I did not encounter any other strings. The rest is English as expected 👍🏻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working (correctly). fixed in upcoming release Fixed in the upcoming release!
Projects
None yet
Development

No branches or pull requests

2 participants