Skip to content

Make few adjustments to [p]mywarnings and its docs#6673

Merged
Kowlin merged 1 commit intoCog-Creators:V3/developfrom
Jackenmen:refactor_mywarnings
Mar 4, 2026
Merged

Make few adjustments to [p]mywarnings and its docs#6673
Kowlin merged 1 commit intoCog-Creators:V3/developfrom
Jackenmen:refactor_mywarnings

Conversation

@Jackenmen
Copy link
Member

Description of the changes

Look at the diff with "Hide whitespace changes" checked to see the changes more accurately.

In summary:

  • fixed incorrect command in docs
  • updated config call to not use the context manager since we don't intend to modify the data
  • updated "You have no warnings" case to have a try-except for Forbidden on DM send and tick only after DM succeeds
  • updated ctx.bot.send_interactive() call to pass user as channel directly instead of calling user.create_dm()
    • presumably this was done like that because there was a period of time when ctx.bot.send_interactive() had a bug that did not allow this
  • removed unnecessary whitespace between description and newline

Have the changes in this PR been tested?

Yes

@Jackenmen Jackenmen added this to the 3.5.23 milestone Mar 4, 2026
@Jackenmen Jackenmen added Type: Bug Unexpected behavior, result, or exception. In case of PRs, it is a fix for the foregoing. Type: Enhancement Something meant to enhance existing Red features. Changelog Entry: Skipped Changelog entry for this PR is unnecessary. labels Mar 4, 2026
@github-actions github-actions bot added the Category: Cogs - Warnings This is related to the Warnings cog. label Mar 4, 2026
@Kowlin Kowlin merged commit 1bfe2a7 into Cog-Creators:V3/develop Mar 4, 2026
20 checks passed
@Jackenmen Jackenmen deleted the refactor_mywarnings branch March 4, 2026 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Category: Cogs - Warnings This is related to the Warnings cog. Changelog Entry: Skipped Changelog entry for this PR is unnecessary. Type: Bug Unexpected behavior, result, or exception. In case of PRs, it is a fix for the foregoing. Type: Enhancement Something meant to enhance existing Red features.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants