-
-
Notifications
You must be signed in to change notification settings - Fork 480
Detect when a game is missing and suggest the user a fix #1374
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
Comments
I am including this handler on my pr #1363 |
@CommandMC @imLinguin any opinion on that? |
I think just showing the 2nd one is enough |
I think it's ok to go with 2 dialogs if it's easier, it's an extra click but it's not a big deal (I would say just 1 dialog if possible, but I don't have a strong opinion) one comment on the first message, showing the app name might be confusing (like... what is stargazer for the end user? I know it's the app_name for magic arena), maybe it was just a quick test and you are showing the actual game title there, in that case ignore this comment haha |
Yes, I changed it to show only the second dialog with the game name. I also think it's easier to understand. |
I'll try to look at it later, I've been having a super busy week, but that PR seems like a really good UX improvement, error handling is really important |
Problem description
There are cases where Heroic will think a game is installed when it actually isn't:
installation finished
when it actually didn't install the gameWhen that happens, trying to run the game will fail saying the directory does not exist, but it shows up as installed and the solution is to either edit the installed games cache file or do some manual work with directories to be able to
uninstall
the game to install it again.Feature description
When a user tries to launch a game, if the install directory is missing, we can tell that to the user and suggest to options to execute:
So the user can try a fix by clicking a button without having to ask for help and touching config files manually.
Alternatives
Do that process manually, that's what has to be done currently.
Additional information
No response
The text was updated successfully, but these errors were encountered: