Skip to content

Feature - Implement warden commands for Foxhole#44

Merged
SyniRon merged 9 commits into7Cav:developfrom
JensGryspeert:feature/implement-warden-commands-foxhole
Jan 12, 2026
Merged

Feature - Implement warden commands for Foxhole#44
SyniRon merged 9 commits into7Cav:developfrom
JensGryspeert:feature/implement-warden-commands-foxhole

Conversation

@JensGryspeert
Copy link

Based on the following received description is this PR created.

Eh, who would be able to review the viability of adding a couple slash commands limited to a role.

I'd like to expand a bit on foxhole support -- I'd like to see if we could get a few /commands added to do three or four things

minimum
/warden add discordname --> add role 'Warden Verified' to user
/warden remove discordname --> remove role 'Warden Verified' from user
/warden purge --> remove all 'Warden Verified' role from all users

stretch
/warden bulkadd comma-separated discord names --> iterate through list, add role 'Warden Verified' to users

and have the use of these limited to either a Warden Admin role, that we'd likely manually assign.

Copy link
Contributor

@SyniRon SyniRon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good other than the one requested change to remove hardcoded role permission. Great work!

@JensGryspeert JensGryspeert force-pushed the feature/implement-warden-commands-foxhole branch from 4603d84 to 9215064 Compare January 11, 2026 23:52
@JensGryspeert JensGryspeert requested a review from SyniRon January 12, 2026 00:19
Copy link
Contributor

@SyniRon SyniRon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@JensGryspeert JensGryspeert requested a review from SyniRon January 12, 2026 08:34
@SyniRon SyniRon merged commit e29445b into 7Cav:develop Jan 12, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants