First of all thank you for your contribution to Vortex!
I can see that you've opted to throw an error during the setup stage instead of raising the dialog when the mod bypass tool is missing.
I'm assuming it's because you were getting an error stating that the `api` argument is undefined.
You can fix that by changing your setup function in the game registration window to this -> `setup: (discovery) => prepareForModding(discovery, context.api),`
Unfortunately throwing an error during the setup stage will immediately fail the game management/activation which is not acceptable UX as not all of the users read the notifications. Can you please use the dialog instead; I'll re-review the code once you upload a new version of your extension that includes this fix, and will add it to the Vortex manifest once I confirm it works as expected.
3 comments
First of all thank you for your contribution to Vortex!
I can see that you've opted to throw an error during the setup stage instead of raising the dialog when the mod bypass tool is missing.
I'm assuming it's because you were getting an error stating that the `api` argument is undefined.
You can fix that by changing your setup function in the game registration window to this -> `setup: (discovery) => prepareForModding(discovery, context.api),`
Unfortunately throwing an error during the setup stage will immediately fail the game management/activation which is not acceptable UX as not all of the users read the notifications. Can you please use the dialog instead; I'll re-review the code once you upload a new version of your extension that includes this fix, and will add it to the Vortex manifest once I confirm it works as expected.
-Nagev