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 two new commands to init script #112

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

jketcham
Copy link
Contributor

I've added two new commands to the init script: "whitelist reload" and "clear."

Whitelist reload is to just reload a server's whitelist while the server is running, which I don't believe was happening if you added or removed a player from the whitelist while the server was running.

The clear command is to clear a players inventory completely or just of a certain entity.

Since this is my first real contribution I hope everything is correct, I'd love pointers on this to get it working!

@marcuswhybrow
Copy link

I've had a look, and I'll merge in the changes when I get a chance to do so.

My instinct tells me that the whitelist reload command is not necessary, since when the server is running, the console `/whitelist add craftysaurus' would be used, and that circumvents the need to reload from file. However we could definitely add it as a useful command.

@jketcham
Copy link
Contributor Author

Ahh I see, I wasn't sure if when you add someone to the white list while the server is running they'd be able to join right away. Makes sense now though.

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.

2 participants