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

handle required mod downloadlinks #146

Closed
wants to merge 2 commits into from

Conversation

uniboi
Copy link
Contributor

@uniboi uniboi commented Apr 14, 2022

broadcasts links to the masterserver and makes them available for the client
depends on R2Northstar/NorthstarMasterServer#58 and R2Northstar/NorthstarMods#309

broadcasts links to masterserver and makes them available for the client
@btarg
Copy link

btarg commented Apr 26, 2022

This would be perfect! I hate having to manually install everything and then manually uninstall/disable when I play on another server.

@uniboi
Copy link
Contributor Author

uniboi commented Apr 26, 2022

@iCrazyBlaze This PR doesn't implement automatic mod downloads. Only a menu is shown that links to the download pages missing mods have specified. If you're looking for automatic mod enabling/disabling, I made another PR for that here: R2Northstar/NorthstarMods#298

Copy link
Member

@EladNLG EladNLG left a comment

Choose a reason for hiding this comment

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

Clean code

10/10 would merge if I could

Also, resolve merge conflicts on the other northstarlauncher PR :3

@uniboi
Copy link
Contributor Author

uniboi commented Nov 1, 2022

succeeded by automatic mod downloads that will come at some point in this century

@uniboi uniboi closed this Nov 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants