-
Notifications
You must be signed in to change notification settings - Fork 0
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 update feature to admin panel #118
Comments
The design decision to only allow one version of a JMOD could also simplify how JMODs are accessed. Right now, JMODs downloaded from Github could have a near infinite number of {
"jmods": {
"github.com/author/jmod": {
"commit": "abcdef",
"maybeOtherFields": "here",
"config": {
"stuff": "here"
}
}
}
} This kind of design would allow for more control over JMOD behavior like limiting the number of threads a JMOD can use |
Big whoopsie, with the new config format, there is a really weird recursive config effect that occurs when trying to update the JMOD config. |
Right now, there is no quick way to update a JMOD. This process should be made as easy and simple as possible to encourage users to update. A decision on how to handle JMOD versions must be made in order to implement this feature. Personally, I see no reason to allow for multiple versions of the same JMOD to exist at the same time (when downloaded from GitHub). This greatly simplifies the upgrade process by keeping the same data directory and passing it to the new JMOD version. Since not all updates may progress smoothly, I may create a method for letting users pin a JMOD to a specific version or tag.
A sample upgrade process may look like this:
The default argument for update would be
latest
and would follow the above process. Alternate arguments would be the tag of the version the user wants to update to. On the admin panel, it may be better to just have users enter the JMOD name with a new version argument instead of having a button for each JMOD. That way, installing and updating JMODs is done by the same method.The text was updated successfully, but these errors were encountered: