AAA3A-AAA3A / AAA3A-cogs

Many cogs for Red-DiscordBot.
https://aaa3a-cogs.readthedocs.io/en/latest
Other
35 stars 28 forks source link

[TempRoles] Allow `[p]temprole add` to overwrite the original duration if higher than remaining. #36

Closed emshov74 closed 1 year ago

emshov74 commented 1 year ago

What cog is this feature request for?

temproles

Describe the feature request in as much detail as possible

The temproles assign command returns a "⚠️ This member already has role" when used and the user already owns this role.

Could this be changed to where the duration of a temprole gets overwritten by the higher value? So for example:

Thank you for your time

AAA3A-AAA3A commented 1 year ago

Hello,

I'm thinking of adding this feature, but maybe with a confirmation before overriding?

Thanks for your suggestion, Have a nice day, AAA3A

emshov74 commented 1 year ago

Maybe make it default to prompt for confirmation but also make it if you enter something like a "noconfirm" flag in the command that it just overwrites? Would be a nice thing for custom commands and the like

AAA3A-AAA3A commented 1 year ago

Or make a [p]temproles edit [confirmation=False] <member> <role> <new_duration> command? [p]temprole add would call it if the temprole have to be override.

emshov74 commented 1 year ago

I'm fine with whatever you can get to work, since I am a very bad programmer :))

AAA3A-AAA3A commented 1 year ago

I've added [p]temproles edit and a confirmation to edit in [p]temproles assign. Ty for your suggestion.

image image