-
-
Notifications
You must be signed in to change notification settings - Fork 94
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
Use DM_MESSAGES instead of PRIVATE_MESSAGES in documentation #1874
Conversation
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.
Really nice catch! I'm surprised it went unnoticed for so long!
Everything looks good to me :)
Looking at the CI, it seems like the fragment changelog is missing. You can read on how to add it here |
Went ahead and added the changelog fragment myself. Thanks again for the pull request! |
Thank you VSCode
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.
Hopefully the final approval!
Summary
PRIVATE_MESSAGES got renamed to DM_MESSAGES 4 years ago :-)
Checklist
nox
and all the pipelines have passed.Related issues
#227