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.

Sign up for GitHub

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

Jump to bottom

Fomod install fixes #617

Merged
merged 3 commits into from
Sep 7, 2023
Merged

Fomod install fixes #617

merged 3 commits into from
Sep 7, 2023

Conversation

Al12rs
Copy link
Contributor

@Al12rs Al12rs commented Sep 7, 2023

  • Fix Fomod analyzer failing if directory of the referenced images is missing.
  • Fix Fomod CLI installer clearing selections of other groups when dealing with "At most one" cases.
  • Fix broken validation check on back operation due to missing Required options at that stage.
  • Fix SkipAll wrongly adding selections for AtLeastOne groups that already had preselected stuff.
@Al12rs
Fix Fomod CLI install deselecting all options in certain cases
fd5c140
@Al12rs
Fix Fomod Analyzer faling to analyze config file if DirectoryNotFound…
62bed17 
… for images.
@Al12rs
- Fix Validation check on Back operation (required options were missing)
00d0980 
- Fix SkipAll to only add options in cases where needed.
- Add required options at the start.
- Don't remove required options on other selections.
@Al12rs Al12rs requested a review from erri120 September 7, 2023 15:07
@Al12rs Al12rs self-assigned this Sep 7, 2023
@halgari halgari merged commit 702ff3b into main Sep 7, 2023
3 of 5 checks passed
@halgari halgari deleted the fomod-install-fixes branch September 7, 2023 22:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants
@Al12rs @halgari