Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Massrole module #56

Merged
merged 30 commits into from
May 1, 2022
Merged

Massrole module #56

merged 30 commits into from
May 1, 2022

Conversation

hfgd123
Copy link
Contributor

@hfgd123 hfgd123 commented Apr 30, 2022

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)

@SCDerox
Copy link
Member

SCDerox commented Apr 30, 2022

sorry, your pull request removes dependencies that are needed in a production enviroment. Please undo these changes and fix the conflicted file package-lock.json.

@SCDerox SCDerox added the enhancement New feature or request label Apr 30, 2022
@SCDerox SCDerox self-requested a review April 30, 2022 20:57
hfgd123 added 5 commits April 30, 2022 23:16
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
@hfgd123
Copy link
Contributor Author

hfgd123 commented May 1, 2022

Hereby I request another review over this module. It should work now I guess

Copy link
Member

@SCDerox SCDerox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@hfgd123 hfgd123 requested a review from SCDerox May 1, 2022 15:50
Copy link
Member

@SCDerox SCDerox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

@hfgd123 hfgd123 requested a review from SCDerox May 1, 2022 17:15
Copy link
Member

@SCDerox SCDerox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One typo away from merging ^^

Co-authored-by: Simon <sc.de@gmx.de>
@SCDerox SCDerox self-requested a review May 1, 2022 17:45
Copy link
Member

@SCDerox SCDerox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for your contribution, your module will be availible on SCNX shortly.

@SCDerox SCDerox merged commit 24c0bd2 into ScootKit:main May 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants