Skip to content

Fix message converter#974

Merged
jchristgit merged 3 commits into
mainfrom
fix-message-converter
Dec 12, 2021
Merged

Fix message converter#974
jchristgit merged 3 commits into
mainfrom
fix-message-converter

Conversation

@ChrisLovering
Copy link
Copy Markdown
Member

@ChrisLovering ChrisLovering commented Dec 5, 2021

Description

Ported from python-discord/bot#1990

Discord.py's message converter is supposed to infer channelID based on ctx.channel if only a messageID is given. A refactor (linked below) a few weeks before d.py's archival broke this, so that if only a messageID is given to the converter, it will only find that message if it's in the bot's cache.

Did you:

…text

Discord.py's message converter is supposed to infer channelID based on ctx.channel if only a messageID is given. A refactor (linked below) a few weeks before d.py's archival broke this, so that if only a messageID is given to the converter, it will only find that message if it's in the bot's cache.
Since w epatched the message converter to work as intended, the help message given to a user when failing to resolve a message reference to a message object has been updated.
@ChrisLovering ChrisLovering added area: backend Related to internal functionality and utilities category: utilities Related to utilities status: needs review Author is waiting for someone to review and approve type: bug Something isn't working labels Dec 5, 2021
Copy link
Copy Markdown
Contributor

@onerandomusername onerandomusername left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Copy Markdown
Contributor

@jchristgit jchristgit left a comment

Choose a reason for hiding this comment

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

I trust Chris.

@jchristgit jchristgit enabled auto-merge December 11, 2021 12:48
@jchristgit jchristgit merged commit 9202b43 into main Dec 12, 2021
@jchristgit jchristgit deleted the fix-message-converter branch December 12, 2021 21:06
@Xithrius Xithrius removed the status: needs review Author is waiting for someone to review and approve label Dec 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: backend Related to internal functionality and utilities category: utilities Related to utilities type: bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants