Conversation
This reverts commit 15bd079.
Fixed remove-all
Removed config.json
Fixed SCDerox request and added anoter TODO
|
sorry, your pull request removes dependencies that are needed in a production enviroment. Please undo these changes and fix the conflicted file |
I think this happened when we tried breaking it to get it to work on windows. ~~I hope it doesn't destroy my module~~
Fixed ESLint syntax errors
Removed semicolon
|
Hereby I request another review over this module. It should work now I guess |
SCDerox
left a comment
SCDerox
left a comment
There was a problem hiding this comment.
Thanks for your contribution! I have marked some things, feel free to discuss them if needed.
I haven't tested it yet, but will once you commit changes.
SCDerox
left a comment
SCDerox
left a comment
There was a problem hiding this comment.
Everything works fine, thanks for your changes! Only one small issue: If the bot can not remove/add a role, the command fails with an exception, please fix that if possible
SCDerox
left a comment
SCDerox
left a comment
There was a problem hiding this comment.
One typo away from merging ^^
Co-authored-by: Simon <sc.de@gmx.de>
SCDerox
left a comment
SCDerox
left a comment
There was a problem hiding this comment.
Thank you for your contribution, your module will be availible on SCNX shortly.
2 participants
Hi. I created a now module, which I think is finished now, so I would like to see it being usable in your service soon. (Only if there are no problems with it of course. Feel free to suggest corrections if you find any)