Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix replies in dms not being sent #3239

Merged
merged 3 commits into from
Mar 14, 2023

Conversation

RealCyGuy
Copy link
Member

In discord.py v2, a new message type discord.MessageType.reply was added so we have to check for that as well.

Fixes #3238

@RealCyGuy RealCyGuy marked this pull request as ready for review January 7, 2023 18:39
@StephenDaDev
Copy link
Member

@Taaku18 If you could look at this as well its a pretty big issue

@StephenDaDev StephenDaDev requested a review from Taaku18 March 13, 2023 23:35
Copy link
Collaborator

@Taaku18 Taaku18 left a comment

Choose a reason for hiding this comment

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

Looks good, will merge now.

@Taaku18 Taaku18 merged commit f2a434b into modmail-dev:development Mar 14, 2023
@RealCyGuy RealCyGuy deleted the fix-replies branch March 15, 2023 16:52
@Taaku18 Taaku18 mentioned this pull request Jul 15, 2023
raidensakura pushed a commit to raidensakura/modmail that referenced this pull request Apr 10, 2024
* Fix replies not being sent

* Use a list instead of a set

---------

Co-authored-by: Taku <45324516+Taaku18@users.noreply.github.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants