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

Add ability to upgrade the mods of all profiles #277

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

logistic-bot
Copy link

Closes #270

Also added an indicator to show which profile is currently being worked on. This is currently shown even if only one profile is being update

Closes gorilla-devs#270

Also added an indicator to show which profile is currently being worked
on. This is currently shown even if only one profile is being update
@CLAassistant
Copy link

CLAassistant commented Feb 14, 2023

CLA assistant check
All committers have signed the CLA.

@logistic-bot
Copy link
Author

This is my first contribution to open source, I hope I did everything correctly.

@tippfehlr
Copy link

While testing this I noticed that it tries to upgrade all profiles, even if there are in the same mod folder, which doesn’t make sense because the freshly upgraded files are removed instantly when upgrading the next profile. There needs to be a check so that only profiles that are in another mod directory are upgraded too. That’s the next problem though: We have no way of knowing which profile is supposed to be active in other directories. (#322)

@theRookieCoder planned to add this with #163, so I don’t think it is necessary to update this.

@theRookieCoder
Copy link
Collaborator

While testing this I noticed that it tries to upgrade all profiles, even if there are in the same mod folder, which doesn’t make sense because the freshly upgraded files are removed instantly when upgrading the next profile

Hmm yes valid point.

There needs to be a check so that only profiles that are in another mod directory are upgraded too

I will consider this when implementing the feature. My plan is to warn or not allow the user to add profiles that output to the same directory, and suggest that they use multiple launcher profiles as intended.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Upgrade all profiles ability
4 participants