-
Notifications
You must be signed in to change notification settings - Fork 6
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
created mod setaotw and removeaotw commands #53
base: master
Are you sure you want to change the base?
Conversation
Yo, thanks! It's being a busy week here. Will try to review/test this on the next weekend. |
hey @vzapo lemme share with you how the AotW thing works on our discord server.
That being said, what I thought to make mods life easier is:
(in other words, a command to remove the role from individuals would not be that necessary) |
ahh i see. might help something like this:
|
There's no need for this, cause the "AotW renovation" is manually handled by the AotW maintainers (currently @ikki5 and Flara). Although they try to attend a schedule, they also have RL commitments. Then I prefer to not make RABot do things automatically in a very specific time. It would create a false impression to the players community that the AotW maintainers must meet the RABot's schedule (and, well, we are all volunteers here...). |
Yeah, you're right. I did not think of that, just wanted to help their work a bit :) Thought it would be easier for them not to remove it manually since they have to credit manually. |
Hey guys, think this is ready for review now.
|
* not using aliases (I realized it brings more confusion than "user-friendliness") * giving proper feedback when the command fails * other cleanups
I've made some changes in the setaotw.js, but now I need to go to bed. Later I'll review the removeaotw.js Thanks! |
created 2 commands for aotw role managment
!setaotw @user
- if done by a mod user will give the mentioned user aotw role.!removeaotw @user
- if done by a mod user will remove aotw winner role from the mentioned user.To check if the user giving the right role and if he has the right role to perform the actions I have added in the
.env
config files 2 variables:ROLE_MOD_NAME
andAOTW_ROLE_ID
which are used in both commands.ROLE_MOD_NAME
hold the name of the role mod('mod' example)AOTW_ROLE_ID
is the id of the role('667583344573284384' example