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.

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

Document URL length limits #6531

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

DEVTomatoCake
Copy link
Contributor

@DEVTomatoCake DEVTomatoCake commented Nov 12, 2023

This PR documents the character length limits for URLs (including image URLs) in embeds and components, as well as the limits for name, state, and url when setting an activity.

@Lulalaby
Copy link
Contributor

Lulalaby commented Nov 12, 2023

Please run npm run fix:tables since the added row is not correctly formatted.
Or separate the different types, since it's really long.

The fix is not automatically, it's just a check whether everything is correct. You still have to do it yourself.

@DEVTomatoCake
Copy link
Contributor Author

Or separate the different types, since it's really long.

Thought about doing so, but as the strings all have the same maximum length imo it's more readable when all of them are combined.

Fixed the tables for now.

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