discord-needle
discord-needle copied to clipboard
fix(rename): limit thread names to less than 100 characters
Fixes #173 by enforcing string limits before renaming the thread.
Do you know if we can use the max_length
option yet with Discord.js? It was added ot the API last week
https://discord.com/developers/docs/change-log#min-and-max-length-for-command-options
Would be better UX knowing in advance!
not sure- don't think so because djs 13.9 isn't out?
discordjs/discord.js#8217 is under v13.9 - IMO, we should wait until then and limit the command itself for better UX.
of course it's already been merged :|
wake me up when djs updates
edit: this didn't age well
@MarcusOtter Should we close in favor of #250 ?
Leave it open for a bit, I will implement these changes manually and add you as a co-author to those commits @c43721
Superseded by #250, added c4 as co-author there (I removed the check as nick suggested)