Allow permissions checking to have a dictionary and not a list at `permissions.json` #267

New issue

Closed

opened 2026-03-09 19:46:09 -04:00 by sirdog3355 · 1 comment

sirdog3355 commented

2026-03-09 19:46:09 -04:00

(Migrated from github.com)

Right now, the permissions.json file looks like this:

{
    "editrole": [
        1364803654825148447
    ],
    "reboot": [
        1362230058462478386
    ],
    "rquote-debug": [
        1362230058462478386
    ],
    "echo": [
        1364803654825148447
    ],
    "blacklist": [ 
        1364803654825148447
    ],
    "trole": [
        1364803654825148447
    ],
    "monitor": [
        1364803654825148447
    ],
    "cl-add": [
        854228249671368705
    ],
    "cl-remove": [
        854228249671368705
    ]
}

Functional. Simple. One problem: Incredibly difficult to keep track of what roles correspond with what numbers.

Desired functionality is to rework permissions checking to allow making these key:value pairs so I can include the role name for readability.

Right now, the `permissions.json` file looks like this: ```json { "editrole": [ 1364803654825148447 ], "reboot": [ 1362230058462478386 ], "rquote-debug": [ 1362230058462478386 ], "echo": [ 1364803654825148447 ], "blacklist": [ 1364803654825148447 ], "trole": [ 1364803654825148447 ], "monitor": [ 1364803654825148447 ], "cl-add": [ 854228249671368705 ], "cl-remove": [ 854228249671368705 ] } ``` Functional. Simple. One problem: Incredibly difficult to keep track of what roles correspond with what numbers. Desired functionality is to rework permissions checking to allow making these key:value pairs so I can include the role name for readability.

sirdog3355 commented

2026-04-03 20:32:08 -04:00

(Migrated from github.com)

After a preliminary assessment of the work this would take, given a variety of factors, I am closing as not planned.