Skip to content

RM-106: Prevent tag from being created if over discord's character limit#59

Merged
BenHall-1 merged 1 commit intoTicketsBot-cloud:masterfrom
biast12:RM-106
Nov 11, 2025
Merged

RM-106: Prevent tag from being created if over discord's character limit#59
BenHall-1 merged 1 commit intoTicketsBot-cloud:masterfrom
biast12:RM-106

Conversation

@biast12
Copy link
Copy Markdown
Contributor

@biast12 biast12 commented Nov 9, 2025

Description

RM-106
Introduces a check to ensure total embed characters do not exceed Discord's 6000 character limit when creating tags.

Also noticed some of the max values where wrong, content got a 2000 character limit (was 4096 before) and the others got a 256 character limit (was 255 before)
https://discord.com/developers/docs/resources/message#create-message
https://discord.com/developers/docs/resources/message#embed-object-embed-footer-structure

image

Type of Change

  • Bug fix
  • New feature
  • Breaking change

Testing

i tried to make a tag with a lot of fields with a lot of text

Checklist

  • My code follows the style of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works

Introduces a check to ensure total embed characters do not exceed Discord's 6000 character limit when creating tags. Also updates max lengths for embed title, author name, and field name to 256, and reduces tag content max length to 2000. Adds TotalCharacterCount method to CustomEmbed for character counting.
@biast12 biast12 requested a review from a team as a code owner November 9, 2025 16:53
@BenHall-1 BenHall-1 merged commit 467725e into TicketsBot-cloud:master Nov 11, 2025
@biast12 biast12 deleted the RM-106 branch November 11, 2025 19:36
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