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.

Sign up for GitHub

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

Jump to bottom

[captcha] show name of role and channel. #32

Merged
merged 6 commits into from
Oct 26, 2023
Merged

[captcha] show name of role and channel. #32

merged 6 commits into from
Oct 26, 2023

Conversation

ltzmax
Copy link
Contributor

@ltzmax ltzmax commented Oct 26, 2023

Here's what i did:

  • Added to show channel and role name by mention.
  • Added missing permission on settings command.
    • it's embed so i added embed permission needed.
  • Added missing min bot version in info.json.
  • Added a reset command.
    • This will reset all settings back to default, instead of going though one by one command.
    • it will need your confirmation before it reset anyhow.

ltzmax added 2 commits October 26, 2023 12:26
@ltzmax
[captcha] show name of role and channel.
61f7278 
* Added missing permission
* Added missing min bot version
* Added a reset command.
@ltzmax
Only one time run.
7a27731
@sourcery-ai sourcery-ai bot mentioned this pull request Oct 26, 2023
Closed
@japandotorg
Copy link
Owner

Please make the changes made by sourcery to close that PR!

@ltzmax
Copy link
Contributor Author

ltzmax commented Oct 26, 2023

Please make the changes made by sourcery to close that PR!

Done

@japandotorg
Copy link
Owner

LGTM! Thanks for the PR!

@japandotorg japandotorg merged commit 3332f4a into japandotorg:main Oct 26, 2023
2 checks passed
@ltzmax ltzmax deleted the captcha branch October 26, 2023 10:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ltzmax @japandotorg