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

Don't throw error on WASM install when MSFS2024 isn't installed #1917

Merged
merged 2 commits into from
Jan 15, 2025

Conversation

neilenns
Copy link
Contributor

Fixes #1916

  • Return true if the community folder isn't found instead of false
  • Get rid of unnecessary parameter since the check happens elsewhere
  • Make sure logs happen when one, but not both, are skipped due to up-to-date

@neilenns neilenns requested a review from DocMoebiuz as a code owner January 15, 2025 01:13
@neilenns
Copy link
Contributor Author

The problem is HandleWasmInstall returns false if the community folder isn't there, which means the Update2024Successful becomes false and we throw the error.

Making it return true in this case is sufficient and makes sure we don't throw the error incorrectly.

Copy link

Build for this pull request:
MobiFlightConnector.zip

}
else
{
Log.Instance.log($"WASM module for MSFS2020 is already up-to-date.", LogSeverity.Info);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will this also be logged even if the user doesn't have 2020 installed?

}
else
{
Log.Instance.log($"WASM module for MSFS2024 is already up-to-date.", LogSeverity.Info);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will this also be logged even if the user doesn't have 2024 installed?

@DocMoebiuz DocMoebiuz merged commit 624cd1e into MobiFlight:main Jan 15, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

WASM install reports failure if only 2020 is installed
2 participants