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.

Sign up for GitHub

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

Jump to bottom

Allow overriding server.properties (re. # 327) #353

Merged

edewit merged 1 commit into OASIS-learn-study:develop from vorburger:OVERRIDE_SERVER_PROPERTIES

Apr 13, 2022

Member

vorburger commented Apr 13, 2022 •

edited

Loading

@edewit This OK for you - merge? I think this is one of the steps for #327 before we can merge your #348.

PS: Next I'll see how hard it would be to do #276 to introduce LuckPerms to see if that could help sort out this other point:

The Player that mineflayer uses doesn't have rights to execute /make


          Allow overriding server.properties (re. # 327)

vorburger requested a review from edewit

April 13, 2022 18:37

vorburger mentioned this pull request

initial version of the mineflayer based npm test #348

Merged

edewit approved these changes

View reviewed changes

edewit merged commit 3c65975 into OASIS-learn-study:develop

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet