-
Notifications
You must be signed in to change notification settings - Fork 43
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
Update guided installer abstraction and FOMOD installer #541
Conversation
This PR conflicts with |
4668c29
to
4969f78
Compare
145a5b1
to
6941924
Compare
This PR doesn't conflict with |
I tested installing a mod from CLI locally, and I found that the experience was rather confusing. Would it be possible to print a description of the expected operation that the user needs to take depending on the type of option group? (ExactlyOne, AtMostOne, AtLeastOne, Any) Like: "Select at least one of the following options" |
The CLI shouldn't be different from |
Always has been NexusMods.App/src/NexusMods.CLI/CliOptionSelector.cs Lines 12 to 16 in 1a21c5d
|
This PR conflicts with |
7b64a32
to
1c3cddb
Compare
This PR doesn't conflict with |
7832544
to
c81b9d1
Compare
Followup on #532 and part 2 of #316. Blocked by #540 and #542.
Fixes #544.
Changes:
IOptionSelector
. We now have theIGuidedInstaller
interface and more internal types that aren't directly related to FOMODs.NexusMods.Games.FOMOD.UI
that isn't complete yet, but will be worked on in the next PR.Originally, I wanted to implement the required ViewModels for the FOMOD UI. However, I had to rework some of our existing FOMOD code because the API of the library we're using is pretty bad.