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

Allow user to start work from DMs. #267

Merged
merged 4 commits into from
Jan 3, 2023

Conversation

AlexanderHott
Copy link
Collaborator

This also broadcasts task completion messages to every server. This needs to be changed in the future if the bot starts sending too many messages, but it should be fine right now.

Copy link
Collaborator

@yk yk left a comment

Choose a reason for hiding this comment

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

LGTM, just a tiny comment

@yk yk linked an issue Jan 2, 2023 that may be closed by this pull request
Copy link
Collaborator

@andreaskoepf andreaskoepf 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 adding the DM prefix command!

@andreaskoepf andreaskoepf enabled auto-merge (squash) January 2, 2023 20:28
@andreaskoepf andreaskoepf requested a review from yk January 2, 2023 20:29
Copy link
Collaborator

@yk yk 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!

@yk
Copy link
Collaborator

yk commented Jan 2, 2023

@AlexanderHott could you have a final look at the merge conflicts in work.py ?

@andreaskoepf andreaskoepf merged commit 29ffc27 into LAION-AI:main Jan 3, 2023
@AlexanderHott AlexanderHott deleted the fix/start-work-from-dm branch January 4, 2023 01:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

I should be able to ask the bot for work in DMs
4 participants