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

Handle long names gracefully #156

Open
FarisZR opened this issue Aug 16, 2024 · 1 comment
Open

Handle long names gracefully #156

FarisZR opened this issue Aug 16, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@FarisZR
Copy link

FarisZR commented Aug 16, 2024

Currently, if a user has a very long name on matrix (or another bridged platform) the discord bridge will reply with this error ⚠ Your message was not bridged: 50035: Invalid Form Body

It would be great if the bridge could just automatically trim names to the maximum length allowed by discord, maybe notify the user on their first message in the chat room about this.

@FarisZR FarisZR added the enhancement New feature or request label Aug 16, 2024
@TheAwiteb
Copy link

TheAwiteb commented Aug 16, 2024

maybe notify the user on their first message in the chat room about this.

I think this will just make it more complicated, Discord allows a maximum of 32 and a minimum of 2 for display name length. So I think a display name longer than 32 doesn't make sense, so using the first 32 characters is enough.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Development

No branches or pull requests

2 participants