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

Add %^compare role/user command #45

Open
Chew opened this issue Dec 2, 2020 · 2 comments · May be fixed by #66
Open

Add %^compare role/user command #45

Chew opened this issue Dec 2, 2020 · 2 comments · May be fixed by #66
Assignees

Comments

@Chew
Copy link
Member

Chew commented Dec 2, 2020

Spontaneous idea. What if there was a way to compare roles and users?

Role compare:

Compare permissions, position, etc. See what one role has and vice versa.

User compare:

Compare age, roles, etc.

@ArChrisVa
Copy link

Hey @Chew i think i can help with this new command , i have already made some minor changes in role comparisons, and i am currently working in the users comparison. I might also think of something new to add in roles comparison, like a better display of color or maybe showing the common role permissions (Let me know if you like any of those 2 suggestions). Your implementation in roles tho seems to be very complete!

@Chew
Copy link
Member Author

Chew commented Jan 22, 2021

Thank you for your contributions!
My current progress is on the command/diff branch, I believe, as I assume you know. There's really no point in showing if they have in common for role comparisons. It may be worth putting all the permission back in the %^roleinfo command (it just shows what the everyone role doesn't have, but this can be replaced by comparing the everyone role with the specified role now)

Feel free to toss a PR targeting that branch, and I'll be happy to review and merge it in! Feel free to hop on the Discord server if you haven't already.

@Chew Chew linked a pull request Feb 13, 2021 that will close this issue
@Chew Chew self-assigned this Feb 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

2 participants