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

Bug Report: Message about missing dotnet-desktop is not translated #54

Closed
1 task done
bovirus opened this issue Nov 28, 2023 · 2 comments
Closed
1 task done

Bug Report: Message about missing dotnet-desktop is not translated #54

bovirus opened this issue Nov 28, 2023 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@bovirus
Copy link
Collaborator

bovirus commented Nov 28, 2023

Is there an existing issue for this?

  • I have searched the existing issues (both open and closed).

What happened or didn't happen?

@Timthreetwelve

If you try to install GetMyIP ina Windows system without dotnet-desktiop appear a window that explain the situation and propose to download dotnet-desktop.

In this window the text is untranslated (it's in english).

Please add relative strings in language file.

Steps To Reproduce

No response

Version

0.9.7

Windows Version

Windows 10 22 H2 64 bit italian

Relevant log output

No response

Anything else?

No response

@bovirus bovirus added bug Something isn't working needs triage labels Nov 28, 2023
@bovirus bovirus changed the title Bug Report: Message about missing donet-desktiop is not translated Bug Report: Message about missing dotnet-desktop is not translated Nov 28, 2023
@Timthreetwelve Timthreetwelve added enhancement New feature or request and removed bug Something isn't working needs triage labels Nov 29, 2023
@Timthreetwelve
Copy link
Owner

@bovirus

The code that detects the missing dotnet desktop runtime is not in my app (any of them). It is most likely built into Windows. So, changing the language in the dotnet dialog isn't an option. (It surprises me that the message is not in the current culture.)

I've been giving this situation some thought, and I wonder if releasing a variant of the app that has all the necessary dotnet files included would be a solution. In the .NET documentation they refer to this as self-contained. This version would be in addition to the current installer and portable versions.

Another option would be to have the Inno Setup script check to see if the required dotnet runtime environment exists before installing. This repository looks like what would be needed. You know Inno Setup better than I do, take a look at it, and let me know what you think.

Thanks,
Tim

@Timthreetwelve
Copy link
Owner

@bovirus

Version 0.9.8 was released a few minutes ago. I realize that it doesn't directly address your issue. However, with the inclusion of the self-contained builds, I believe that it is a good workaround.

The addition of the self-contained builds required some updates to the Inno Setup script and the PowerShell script that invokes the Inno Setup compiler. I've added a repo here for the PowerShell script. I added GetMyIPEx.iss which is the script used going forward.

I'm sorry for the delay and lack of communication recently, I had another health setback and was hospitalized for a few days. I'm back home now and feeling better. The worst part was the hospital Wi-Fi was down!

Tim

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants