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

[Mod Manager] Improvements to Collection Import #317

Merged
merged 5 commits into from
Sep 29, 2023

Conversation

Dyvinia
Copy link
Contributor

@Dyvinia Dyvinia commented Sep 26, 2023

  • Improvements to Collection import
    • Collections can now be imported even if they are not in an archive
    • Alerts user if trying to drag in mod files from inside of an archive (such as drag/drop from winrar/7zip etc

@Dyvinia Dyvinia changed the title check for dll files when installing mods using a zip file [Mod Manager] Improvements to Mod/Collection Import Sep 27, 2023
@Dyvinia
Copy link
Contributor Author

Dyvinia commented Sep 27, 2023

Had to put the 2 changes into one PR since they both edit the same area

@Dyvinia Dyvinia changed the title [Mod Manager] Improvements to Mod/Collection Import [Mod Manager] Improvements to Collection Import Sep 27, 2023
{
string[] filenames = (string[])e.Data.GetData(DataFormats.FileDrop, true);
InstallMods(filenames);

ICollectionView view = CollectionViewSource.GetDefaultView(availableModsList.ItemsSource);
view.Refresh();
}
else if (e.Data.GetFormats().Any(f => f == "FileContents"))
Copy link
Collaborator

Choose a reason for hiding this comment

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

does that not write them to the tmp storage and then u should be able to import them anyways?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

windows zip doesnt, winrar however does and it will satisfy the first if statement

@wannkunstbeikor wannkunstbeikor merged commit f11f3eb into CadeEvs:1.0.6.3 Sep 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants