Skip to content

Refactor send_message() and send_error() #7

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

Merged
merged 1 commit into from
Jul 2, 2025

Conversation

Endkind
Copy link
Member

@Endkind Endkind commented Jul 2, 2025

  • Deprecated 'msg' parameter in send_message() and send_error(), but it's still functional for compatibility.
  • send_message() now fully supports everything interaction.response.send_message() can do.
  • send_message() checks if the interaction is still valid; if not, it logs instead of sending a message.
  • send_error() now sends an Embed instead of plain text.
  • send_error() supports 'title' and 'description' for the Embed, with a default title if none is provided.

- Deprecated 'msg' parameter in send_message() and send_error(), but it's still functional for compatibility.
- send_message() now fully supports everything interaction.response.send_message() can do.
- send_message() checks if the interaction is still valid; if not, it logs instead of sending a message.
- send_error() now sends an Embed instead of plain text.
- send_error() supports 'title' and 'description' for the Embed, with a default title if none is provided.
@Endkind Endkind self-assigned this Jul 2, 2025
@Endkind Endkind added the feature New functionality or enhancements label Jul 2, 2025
@Endkind Endkind merged commit 92fd3be into master Jul 2, 2025
1 check passed
@Endkind Endkind deleted the change-send_error-style branch July 2, 2025 08:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New functionality or enhancements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant