Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Update docs/docstring to make [p]modlogset's new home more clear #6182

Closed
Flame442 opened this issue Jun 11, 2023 · 0 comments · Fixed by #6300
Closed

Update docs/docstring to make [p]modlogset's new home more clear #6182

Flame442 opened this issue Jun 11, 2023 · 0 comments · Fixed by #6300
Labels
Category: Cogs - Modlog This is related to the Modlog cog. Good First Issue Good for beginners to the project. Status: In Progress There's a PR open to deal with this, or an org member is working on this internally Type: Enhancement Something meant to enhance existing Red features.

Comments

@Flame442
Copy link
Member

Now that [p]modlogset has been moved to be under core_commands.py with 3.5, it is somewhat unclear that this command exists when looking at the help output for the Modlog cog and the Modlog docs. A reference to this command should probably be put in the Modlog cog. I would guess that best place for this to go is probably the cog's docstring.

This change might also make sense for [p]bankset and the Economy cog, though I think that one is a bit less confusing.
@Flame442 Flame442 added Type: Enhancement Something meant to enhance existing Red features. Good First Issue Good for beginners to the project. Status: PRs Welcome No one is currently assigned to this issue, but we'd be grateful if anyone made a PR. Category: Cogs - Modlog This is related to the Modlog cog. labels Jun 11, 2023
@Flame442 Flame442 added Status: In Progress There's a PR open to deal with this, or an org member is working on this internally and removed Status: PRs Welcome No one is currently assigned to this issue, but we'd be grateful if anyone made a PR. labels Mar 2, 2024
@Flame442 Flame442 linked a pull request Mar 2, 2024 that will close this issue
Update ModLog to include info about [p]modlogset #6300
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Category: Cogs - Modlog This is related to the Modlog cog. Good First Issue Good for beginners to the project. Status: In Progress There's a PR open to deal with this, or an org member is working on this internally Type: Enhancement Something meant to enhance existing Red features.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update ModLog to include info about [p]modlogset BlizzardTheWolf/Red-DiscordBot
1 participant
@Flame442