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

Add /help and /tutorial commands and update bot activity #399

Merged
merged 8 commits into from
Jan 6, 2023

Conversation

AlexanderHott
Copy link
Collaborator

@AlexanderHott AlexanderHott commented Jan 5, 2023

  • /help
  • /tutorial

@AlexanderHott
Copy link
Collaborator Author

image

@AlexanderHott AlexanderHott mentioned this pull request Jan 5, 2023
14 tasks
@AlexanderHott
Copy link
Collaborator Author

I need some help writing the /tutorial command. I can style it and everything else, but I'm not sure how specific I should be.

@AlexanderHott AlexanderHott changed the title Add /help command and update bot activity Add /help /tutorial command and update bot activity Jan 5, 2023
@AlexanderHott AlexanderHott changed the title Add /help /tutorial command and update bot activity Add /help, /tutorial, and command and update bot activity Jan 5, 2023
@AlexanderHott AlexanderHott changed the title Add /help, /tutorial, and command and update bot activity Add /help and /tutorial commands and update bot activity Jan 5, 2023
@yk
Copy link
Collaborator

yk commented Jan 5, 2023

I guess we can just make a video tutorial and then link to that

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.

Please show help for admin-commands only to bot-owner users.

if can_manage_guild:
content += f"""\

{_li("**`/settings log_channel <channel>`**")}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you please show help for the admin-commands (configuration, reload etc.) only when the requesting user is a bot-owner? (For ordinary users these commands are irrelevant.)

Copy link
Collaborator

Choose a reason for hiding this comment

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

In the first bot-implementation this was simply done via the templates. I still think that removing the templates was a step backwards.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I agree that templates are probably better.

If someone invites the bot to their server, they should be able to set the channel that the bot sends messages in. The log channel is a server specific configuration, so anyone who can manage the server should be able to use it

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I can switch back to using templates in another PR, but I'll have to look up how to use them async. I think aiofiles might be the easiest way.

andreaskoepf and others added 2 commits January 6, 2023 19:34
@andreaskoepf andreaskoepf enabled auto-merge (squash) January 6, 2023 18:42
@andreaskoepf andreaskoepf merged commit 8970195 into LAION-AI:main Jan 6, 2023
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.

4 participants