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

Added server and client proxies so that the mod doesn't crash on dedicated server #3

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

Conversation

GrumpyChunks
Copy link

I've just used your mod in a modpack I've put together on the Curse Voice client, and understand that this is a client only mod. However, when I put packs together, I always ensure that the pack will work both on a client and dedicated server. It's not possible to maintain server versions of packs on the Curse Voice Client, so I only get a single pack that has to work on both server and client.

I've forked your mod, and have added a sided proxy, such that if the mod is installed on a dedicated server, no initialisation is performed, and the mod isn't loaded, however on a client, the mod will be loaded as normal. I've not made any other changes, so functionality shouldn't have changed.

If you're happy with the changes I've made, I'd be delighted if you'd merge these changes into your code base, then when the mod is next updated, players that use my pack (and other packs that use your mod), will have a nice easy method of getting a dedicated server up and running without having to deal with exceptions that they probably won't understand.

…mod has been installed on a dedicated server
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant