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

Remove reliance on a permissions pluign, support 1.10-1.13, minor tweaks… #1

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

Conversation

Selim042
Copy link

… overall, add the ability to support multiple server versions.

… overall, add the ability to support multiple server versions.
@Selim042
Copy link
Author

This is currently untested, just wanted to get this pulled when I am thinking about it.

@Selim042
Copy link
Author

Selim042 commented Feb 1, 2018

This has been tested now, should be fully functional.

@Selim042 Selim042 changed the title Remove reliance on a permissions pluign, update to 1.12, minor tweaks… Remove reliance on a permissions pluign, update to 1.12 (and 1.13), minor tweaks… Jul 29, 2018
@Selim042
Copy link
Author

Should now support 1.13, closes #2 since the error reported there isn't the cause of the source code here and logged from my addition. Sending jar to OP of the issue for testing.

@Resoluciones
Copy link

Hi! I can confirm it's perfectly working in 1.13, now, thanks to Selim!

@Selim042
Copy link
Author

Selim042 commented Jul 29, 2018

Cool! Thanks for testing.

@Selim042 Selim042 changed the title Remove reliance on a permissions pluign, update to 1.12 (and 1.13), minor tweaks… Remove reliance on a permissions pluign, support 1.10-1.13, minor tweaks… Jul 30, 2018
@Selim042
Copy link
Author

Hey, @ltguide any chance this could get merged? Or, if you'd rather not manage this project anymore I could push updates whenever Mojang releases Minecraft updates. With your permission of course.

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