-
-
Notifications
You must be signed in to change notification settings - Fork 480
Add 'suppress' to messageable.send() #1587
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
Conversation
Co-authored-by: Dorukyum <53639936+Dorukyum@users.noreply.github.com>
Co-authored-by: Dorukyum <53639936+Dorukyum@users.noreply.github.com>
Co-authored-by: Dorukyum <53639936+Dorukyum@users.noreply.github.com>
Co-authored-by: Dorukyum <53639936+Dorukyum@users.noreply.github.com>
Co-authored-by: Dorukyum <53639936+Dorukyum@users.noreply.github.com>
Co-authored-by: Emre Terzioglu <50607143+EmreTech@users.noreply.github.com>
Co-authored-by: Emre Terzioglu <50607143+EmreTech@users.noreply.github.com>
|
Got when trying to use this with a thread, using version 2.1.3. It's late night here, so just noting it down as a comment. I'll try to verify and reproduce on a minimal example when I'm fresh and have more time. If I'm successful, I'll get back filing a separate issue. |
This change isn't in 2.1.3 |
|
Oh! That would explain it then. This probably saved me hours of debugging, thank you 😀 I first got confused by docs (using the latest docs, but not the latest code from git means I may see new features not actually present in code), then by the order of the commits in the main branch, which got me thinking that this must be released since 2.1.2: Since |

Summary
Adds support for "suppress" argument for messageable.send(), mirroring message.edit() suppression argument.
Information
Checklist
type: ignorecomments were used, a comment is also left explaining why.