Skip to content
This repository has been archived by the owner on Aug 22, 2023. It is now read-only.

[Feature Request]: Prevent "Modpack is already up to date!" #78

Closed
1 task done
Madis0 opened this issue Feb 8, 2023 · 6 comments
Closed
1 task done

[Feature Request]: Prevent "Modpack is already up to date!" #78

Madis0 opened this issue Feb 8, 2023 · 6 comments
Labels
enhancement New feature or request
Milestone

Comments

@Madis0
Copy link
Member

Madis0 commented Feb 8, 2023

Code of Conduct

  • I agree to follow this project's Code of Conduct

Potential Enhancement

This window pops up by packwiz when the modpack VI is installing is apparently already up to date
image
It is shown for 10 seconds, after which it closes automatically.

Can VI prevent showing it altogether?

How it could be implemented

Some packwiz check to see if things are up to date? Check the folder hash?

The goal is to prevent the install before Packwiz even launches, so that the timer won't count down.

Anything else?

Not a big deal but would be nice to have regardless

@Madis0 Madis0 added the enhancement New feature or request label Feb 8, 2023
@osfanbuff63
Copy link
Member

packwiz-installer has an option to hide the GUI altogether, which I had considered passing, or maybe as a configuration option? The output is logged to the console (which in turn is redirected to the log, or at least should be), so if we need the information from it we can just ask for the logs/vanilla_installer.log. (For some reason, that is being created in the parent directory, but I'm going to make an issue for that

Either way, this is likely possible without too much struggle, as we can probably use hashes to determine updates (say, hash index.toml or pack.toml and compare it to the saved version in the config file (maybe something like debug.index_hash?).

@osfanbuff63 osfanbuff63 added this to the Beta 3 milestone Feb 8, 2023
@comp500
Copy link

comp500 commented Feb 15, 2023

You can pass the --timeout flag to packwiz-installer to bypass the timeout. This does however mean that the user won't be able to reconfigure optional mods if the list of optional mods hasn't changed, unless they manage to click the button very quickly.

@osfanbuff63
Copy link
Member

I don't think FO uses optional mods (at least on stable), so that will be a good fix as well. Thanks ❤️

@comp500
Copy link

comp500 commented Feb 16, 2023

If a pack doesn't have any optional mods, it won't bother waiting at all :)

@Madis0
Copy link
Member Author

Madis0 commented Feb 16, 2023

FYI 1.16.5 does currently have Hydrogen as an optional mod (because it needs exactly Java 8), but the purposes of VI I'd say automatic skipping is a good thing.

@osfanbuff63
Copy link
Member

Well, this will at least prevent the original issue from occurring - you'll still be asked to configure optional mods on 1.16.5 if this is the first time installing it [to this directory], which IMO is a good thing - especially because we don't support 1.16.5 anymore 😅

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants