Skip to content

Bulk editing#285

Merged
Turbotailz merged 11 commits into
masterfrom
feature/bulk-edit
Sep 3, 2020
Merged

Bulk editing#285
Turbotailz merged 11 commits into
masterfrom
feature/bulk-edit

Conversation

@Turbotailz
Copy link
Copy Markdown
Member

@Turbotailz Turbotailz commented Jun 28, 2020

This PR adds bulk editing which allows users to select multiple permission nodes (across any group/user) and lets them copy, move, delete or update the nodes.

image

To do:

  • Copy nodes
  • Move nodes
  • Delete nodes
  • Update nodes
  • Improve performance when groups have large sets of nodes

Visit https://luckperms.turbotailz.com/editor/demo to try it out

Resolves #54
Resolves #86
Resolves #128
Resolves #199
Resolves #262

@lucko
Copy link
Copy Markdown
Member

lucko commented Jun 30, 2020

Had a poke around, looks great!

One small thing I noticed - seems a little laggy when selecting in groups with lots of entries. For example, open admin in the demo data and click the checkboxes to reproduce what I mean (hopefully!)

@Turbotailz
Copy link
Copy Markdown
Member Author

One small thing I noticed - seems a little laggy when selecting in groups with lots of entries. For example, open admin in the demo data and click the checkboxes to reproduce what I mean (hopefully!)

Yeah the admin group has always been a bit laggy in the demo due to how many nodes are present. I will probably look at fixing that in this PR - once I figure out how 😆

@Turbotailz
Copy link
Copy Markdown
Member Author

Fixed performance for large node lists! Will also do this for the verbose viewer 👌

@Turbotailz
Copy link
Copy Markdown
Member Author

Pretty much done most of the work on this now. Keen to get more people testing it out before I merge into production.

@ARR4NN
Copy link
Copy Markdown

ARR4NN commented Aug 26, 2020

This Feature is 100% needed it will make a lot of permissions easier to work with . My 10 mins of playing round with it , it seemed prefect . would love to see this come live one time soon !

@Turbotailz
Copy link
Copy Markdown
Member Author

Merging now as I have not received any other feedback than the comment above :)

@Turbotailz Turbotailz merged commit fcea0c1 into master Sep 3, 2020
@Turbotailz Turbotailz deleted the feature/bulk-edit branch September 3, 2020 00:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants