-
Notifications
You must be signed in to change notification settings - Fork 52
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
Emergency Staff Alert #48
base: master
Are you sure you want to change the base?
Conversation
) | ||
alert_embed.add_field(name="Reason", value=reason, inline=False) | ||
view = EmergencyView(ctx) | ||
view.add_item( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can't this button be part of the class?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To my knowledge the only ways are to either 1) clear_items()
in the __init__
and then add the buttons manually or 2) Have a separate button subclass for the resolve button and then add them in the __init__
manually. This way felt easier to me but definitely can do one of those two (or any other way i may be unaware of), what do you think?
Converted to draft until the role mention problem is fixed |
b2fe6cd
to
25785f5
Compare
Additions
emergency
command for issuing emergency staff alertsemergency ban
staff-only command for banning users from issuing emergency staff alertsemergency unban
staff-only command for unbanning users banned from issuing emergency staff alertsModifications