CloudNet icon indicating copy to clipboard operation
CloudNet copied to clipboard

feat: New plugin made to support LuckPerms

Open RobotHanzo opened this issue 2 years ago • 3 comments

Motivation

In PR #1253 CloudPerms was removed, making LuckPerms one of the best alternatives But it lacks the feature of service / task exclusive permission Therefor an optional plugin was made for those who want to use LuckPerms and use task exclusive permissions

Modification

Added a new plugin named CloudNet-LuckPerms-Addon to add some LuckPerms context (explained below)

Result

image Four types of context were added: task, service, environment and node Server admins may then setup context in their permissions if needed, to make a permission service / task exclusive (Note: I only made the plugins for platforms which both CloudNet and LuckPerms support, and this has been tested on Paper & Velocity)

Other context

N/A

RobotHanzo avatar Jun 16 '23 17:06 RobotHanzo

Done.

RobotHanzo avatar Jun 17 '23 16:06 RobotHanzo

Just want to throw this information here: we're planning on a seperate module to better support luckperms (for example auto downloading for the environments). I think this plugin would fit more into the module (or, we can leave it as a plugin but the LP code will be in a seperate repo anyway).

So this PR won't be merged here, but having that functionality seems like a great addition.

derklaro avatar Jun 17 '23 17:06 derklaro

Just want to throw this information here: we're planning on a seperate module to better support luckperms (for example auto downloading for the environments). I think this plugin would fit more into the module (or, we can leave it as a plugin but the LP code will be in a seperate repo anyway).

So this PR won't be merged here, but having that functionality seems like a great addition.

Alright, I'd be glad to move this feature into the module when it's made

RobotHanzo avatar Jun 17 '23 17:06 RobotHanzo

Hi @RobotHanzo, it took longer than we expected. But currently I am working the last preparations to remove cloudperms. The idea to create a new luckperms module is on ice (at least for now). So I would like to include the plugin you developed.

Can you maybe update this pr to RC10 and the latest minecraft version (1.20.4 is enough for now). If you don't have the time to do that, just leave a message and I will try to update the pr.

0utplay avatar May 18 '24 19:05 0utplay

Hi @RobotHanzo, it took longer than we expected. But currently I am working the last preparations to remove cloudperms. The idea to create a new luckperms module is on ice (at least for now). So I would like to include the plugin you developed.

Can you maybe update this pr to RC10 and the latest minecraft version (1.20.4 is enough for now). If you don't have the time to do that, just leave a message and I will try to update the pr.

Done

RobotHanzo avatar May 18 '24 19:05 RobotHanzo

Please exec gradlew spotlessApply once

0utplay avatar May 18 '24 20:05 0utplay

I just discovered that context suggestions are supported, so I added that (Which I believe adds our contexts to the web panel for easier access)

RobotHanzo avatar May 19 '24 09:05 RobotHanzo

Thanks for the contribution!

0utplay avatar May 19 '24 10:05 0utplay