vrt-cogs icon indicating copy to clipboard operation
vrt-cogs copied to clipboard

Update the roles of individual users through `[p]lvlset roles initialize`

Open cswimr opened this issue 1 year ago • 4 comments

This pull request adds a target argument to the [p]levelset roles initialize command, allowing a server administrator to sync a specific users' level roles. This functionality is complete and works, the reason this PR is currently marked as a draft is because of the guild cooldown on this command. I think the cooldown should only be enforced if the target argument isn't provided, as updating a single user probably isn't a very expensive operation, compared to updating the entire server. However, I wanted to get others' opinions on this before changing it as it is a different change.

cswimr avatar Aug 05 '24 20:08 cswimr

I think the cooldown should only be enforced if the target argument isn't provided You can add logic that resets the cooldown if a target is provided

Also, if you're going to change the embed color, please use the correct English spelling of it (i.e. ctx.embed_color())

vertyco avatar Aug 05 '24 20:08 vertyco

A small change log entry should be included as well please

vertyco avatar Aug 05 '24 20:08 vertyco

@cswimr is this PR still active?

vertyco avatar Sep 20 '24 14:09 vertyco

Oh, yes. I'll resolve the conflicts and finish the functionality in a few days probably when I have a bit more time to spend on it, life's been busy for like the last month lol

cswimr avatar Sep 20 '24 15:09 cswimr

Marking this as closed due to inactivity

vertyco avatar Nov 29 '24 21:11 vertyco