-
-
Notifications
You must be signed in to change notification settings - Fork 17
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
Internal overhaul task #71
Conversation
269f138
to
df3bd83
Compare
All is done except for integrity checking as that would require a lot more changes, currently finished the overhaul but need testing to assure it isn't breaking anything. |
In your fix for #62, do you hide only versions with "custom" release type or all modded versions? (some have release or snapshot build type) Does it affect the "latest version" option? |
Customs are those not in the remote list (version manifest), so those that are not releases, snapshots, old alpha and beta |
e108504
to
564597d
Compare
rebased to include new CI stuff |
564597d
to
6842aaf
Compare
Moved authentication database, selected user and client token out of launcher profiles and into launcher accounts. Fixed issues with JVM Arguments having extra spaces. Fixed issues with game output streams being closed early and made the process "safer".
added latest snapshots latest versions based on remote not local (olauncher#62) cache version manifest for offline use add version filter settings
6842aaf
to
42863dd
Compare
RE: latest commit |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good yay
These are somethings I will do within this overhaul. So far no issues with the new accounts files the other miscellaneous changes. Would be best to test out before I finish.