Merge /editrole and /trole #209

Closed
opened 2026-01-02 22:01:36 -05:00 by sirdog3355 · 1 comment
sirdog3355 commented 2026-01-02 22:01:36 -05:00 (Migrated from github.com)

Right now, /trole allows assigning a role temporarily whereas /editrole allows giving a role permanently. Both commands pull from the same list of allowable roles in data/variables.json.

Desired functionality is for /editrole to be deprecated, /trole be renamed to /role, and if the argument length is set to 0 that the role be given permanently, just like how /editrole does it.

Right now, `/trole` allows assigning a role temporarily whereas `/editrole` allows giving a role permanently. Both commands pull from the same list of allowable roles in `data/variables.json`. Desired functionality is for `/editrole` to be deprecated, `/trole` be renamed to `/role`, and if the argument `length` is set to `0` that the role be given permanently, just like how `/editrole` does it.
sirdog3355 commented 2026-01-27 14:42:01 -05:00 (Migrated from github.com)

This is quite the undertaking. Perhaps I may revisit in the future, but there is no pressing need, and I've got a lot on my plate. Gonna close this as not being done.

This is quite the undertaking. Perhaps I may revisit in the future, but there is no pressing need, and I've got a lot on my plate. Gonna close this as not being done.
Sign in to join this conversation.
No description provided.