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

Feat: Expand menu.ts and let bot handle break role #582

Closed
wants to merge 8 commits into from

Conversation

hasanlatch007
Copy link
Contributor

I split the commands a bit so when @adrian-goe adds the coin feature he would be easily able to implement it.
Currently, this only sets the preparation for the new feature of letting the bot handle the break role
Also added it to its own personal folder for future usage for Adrian's features :)

!closes #196

@hasanlatch007 hasanlatch007 added the enhancement New feature or request label Nov 4, 2021
@hasanlatch007 hasanlatch007 marked this pull request as draft November 4, 2021 21:45
@adrian-goe
Copy link
Member

That's 2 seperate things in one MR.
Also, you need to rebase.

@hasanlatch007 hasanlatch007 force-pushed the feat/let-bot-handle-break-role branch from aec5f2a to 8597f6f Compare November 26, 2021 21:59
@hasanlatch007
Copy link
Contributor Author

hasanlatch007 commented Nov 26, 2021

This requires Yes-Theory-Fam/database#95 to be merged, once it is merged I will update the DB version in the package.json

@hasanlatch007 hasanlatch007 marked this pull request as ready for review November 26, 2021 22:12
Copy link
Member

@mvarendorff mvarendorff left a comment

Choose a reason for hiding this comment

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

[email protected] is available now as well

src/programs/menu/break-handler/addBreak.ts Outdated Show resolved Hide resolved
src/programs/menu/break-handler/addBreak.ts Outdated Show resolved Hide resolved
src/programs/menu/break-handler/addBreak.ts Outdated Show resolved Hide resolved
src/programs/menu/break-handler/addBreak.ts Outdated Show resolved Hide resolved
src/programs/menu/break-handler/addBreak.ts Outdated Show resolved Hide resolved
src/programs/menu/common.ts Show resolved Hide resolved
src/programs/menu/common.ts Outdated Show resolved Hide resolved
src/programs/menu/common.ts Outdated Show resolved Hide resolved
src/programs/menu/dm-menu.ts Outdated Show resolved Hide resolved
src/programs/menu/nameChange.ts Outdated Show resolved Hide resolved
src/programs/menu/break-handler/add-break.ts Outdated Show resolved Hide resolved
src/programs/menu/break-handler/add-break.ts Outdated Show resolved Hide resolved
src/programs/menu/break-handler/remove-break.ts Outdated Show resolved Hide resolved
src/programs/menu/break-handler/remove-break.ts Outdated Show resolved Hide resolved
@hasanlatch007 hasanlatch007 force-pushed the feat/let-bot-handle-break-role branch from 0d2beae to 3cda94b Compare February 20, 2022 14:15
@hasanlatch007 hasanlatch007 requested review from mvarendorff and removed request for adrian-goe February 20, 2022 14:21
@hasanlatch007
Copy link
Contributor Author

Closing this since it is too old, and will need to be updated, better start from scratch.

@mvarendorff mvarendorff deleted the feat/let-bot-handle-break-role branch January 22, 2023 13:41
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.

Let the bot handle the Break role
3 participants