search

CodeMastr3 / hackbot

Discord bot for Hackathon
4 stars 14 forks source link

Fixes #90 and #10 #97

Closed Ramsfield closed 2 years ago

Ramsfield commented 2 years ago

Fixes #10 and #90

Adding and removing roles has been somewhat of a sticky issue with the bot for a while. Originally, users would have to match the case of a role letter for letter. This was then updated so that they were case insensitive, but roles with capitalization would fail to be added (e.g. Announcement). This update will fix #90 such that a role is now associated with it's own lower case version of the role and will be reassociated with the original word.

This update also fixes #10. Sometimes, a user will have a blank space character at the end of a role name. This will henceforth sanitize all the user's inputs so that all blank space chars are removed.

This has been tested as functional.

Ramsfield commented 2 years ago

@thomasdevine01 I would be honored to have you look this over and give your opinion on this.

thomasdevine01 commented 2 years ago

https://www.youtube.com/watch?v=CpjH9M2SYsk

CodeMastr3 commented 2 years ago

Sorry I've been busy I've looked the code over but I haven't ran it to test but it all looks great