Deprecate variable "admin_role_id" in favor of programmatic determination of Administrator permission #175

New issue

Closed

opened 2025-12-11 20:05:00 -05:00 by sirdog3355 · 1 comment

sirdog3355 commented

2025-12-11 20:05:00 -05:00

(Migrated from github.com)

This is used programmatically for /user and /blacklist.

/user

Used by command to denote someone as a staff member. This will no longer be relevant after #174 has a merged PR.

/blacklist

Used to reject command execution on someone who is a staff member. Should replace the checking of the role with whether the person has the Discord administrator permission like how utils/permissions_checker.py does it.

This is used programmatically for `/user` and `/blacklist`. ### /user Used by command to denote someone as a staff member. This will no longer be relevant after #174 has a merged PR. ### /blacklist Used to reject command execution on someone who is a staff member. Should replace the checking of the role with whether the person has the Discord administrator permission like how `utils/permissions_checker.py` does it.

sirdog3355 commented

2025-12-14 23:40:46 -05:00

(Migrated from github.com)

Addressed by #182. No documentation updates necessary. Closing as completed.