Skip to content
This repository has been archived by the owner on Mar 13, 2023. It is now read-only.

feat: support component only messages #630

Merged
merged 2 commits into from
Sep 14, 2022

Conversation

LordOfPolls
Copy link
Member

What type of pull request is this?

  • Non-breaking code change
  • Breaking code change
  • Documentation change/addition
  • Tests change

Description

Adds support for sending messages with only components
discord/discord-api-docs#5435

Changes

  • Add components to the minimum required content in a message send

Checklist

  • I've formatted my code with Black
  • I've ensured my code works on Python 3.10.x
  • I've tested my code

@Kigstn
Copy link
Member

Kigstn commented Sep 9, 2022

Depending on how legible the error is that discord returns when a message is fully empty, it may be worth removing the check completely

@LordOfPolls
Copy link
Member Author

LordOfPolls commented Sep 9, 2022

There arent really any reasonable ways to send messages that would raise an exception. Except send()

I'm removing the check - however ive noticed a bug with error parsing thanks to discord, making a seperate pr for that

@LordOfPolls LordOfPolls added the New Feature::Discord Adding support for a new discord feature label Sep 9, 2022
@LordOfPolls LordOfPolls merged commit ebc3675 into dev Sep 14, 2022
@LordOfPolls LordOfPolls deleted the support-component-only-messages branch September 14, 2022 17:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
New Feature::Discord Adding support for a new discord feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants