-
-
Notifications
You must be signed in to change notification settings - Fork 469
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: message forwarding #2598
base: master
Are you sure you want to change the base?
feat: message forwarding #2598
Conversation
more or less done, just need to add mention/role mention handling and partialmessage support... but still 403ing on my bot 😔 |
unlucky bcs its 95% out. you're in th holdout group. try making a new bot |
Signed-off-by: UK <41271523+NeloBlivion@users.noreply.github.com>
@NeloBlivion status? |
Signed-off-by: UK <41271523+NeloBlivion@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
one changelog please
Head branch was pushed to by a user without write access
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Testing required
ah yeah, forgot to tick that box; as far as i've tested there haven't been any issues. |
from .message import MessageReference | ||
|
||
if not isinstance(reference, MessageReference): | ||
utils.warn_deprecated( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doesn't match the InvalidArgument
error message below, also why?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we discussed this when forwarding came out; before you could pass any Message
into reference
to automatically reply to it, but since forwarding was added this behaviour is too vague and should be removed after deprecation.
Summary
Implements message forwarding, alongside some small cleanups for other message features.
Information
examples, ...).
Checklist
type: ignore
comments were used, a comment is also left explaining why.