Skip to content

[SlashHelpMenu] improvements#230

Merged
ltzmax merged 2 commits intoltzmax:masterfrom
Evanroby:slash-help
Apr 5, 2026
Merged

[SlashHelpMenu] improvements#230
ltzmax merged 2 commits intoltzmax:masterfrom
Evanroby:slash-help

Conversation

@Evanroby
Copy link
Copy Markdown
Contributor

@Evanroby Evanroby commented Apr 5, 2026

well, do I really need to introduce myself ?
fixed a bunch of stuff in slashhelpmenu. Cog names were being mangled by .title() so switched to qualified_name, subcommand mentions always fell back to plain text which is now fixed, embed char count wasn't including title/description so it could silently blow past 6000 chars, and /help was randomly gated behind embed_links. also added red's permission checks (allowed_by_whitelist_blacklist + cog_disabled_in_guild) to _can_use, a close button on the paginator, current settings display on [p]slashhelpset, concurrent mention fetching, get_embed_color instead of hardcoded blurple, and fixed a race condition on view timeout by storing the original interaction instead of view.message.

Changes were tested.

@Evanroby
Copy link
Copy Markdown
Contributor Author

Evanroby commented Apr 5, 2026

ah yes pre-commit, you love being off-topic

@ltzmax
Copy link
Copy Markdown
Owner

ltzmax commented Apr 5, 2026

Pre commit doing a job I didn't do 🤣

@ltzmax ltzmax merged commit 831b40b into ltzmax:master Apr 5, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants