CodeMastr3 / hackbot

Discord bot for Hackathon
4 stars 14 forks source link

Role names with capitals won't get normalized when adding a role #90

Closed Ramsfield closed 2 years ago

Ramsfield commented 2 years ago

Currently we will hard code specific roles to accommodate for a role that might have a capital letter. This is bad practice. We should normalize the role names on both sides before solely trying to get it. A hash map would probably be best suited for that: create a map of role.lower() to role, and use requested_role.lower() to access the role name before asking discord for it.