Slash Commands Integer Option max and min

[itohatweb]

Description
Allow us to define maximum and minimum values for integer options. The number that the executor then returns must be in the range of the specified max and min values.

Why This is Needed
Example: A volume command for music bots. This would allow users to select a volume between 0 and 200 only.

Alternatives Considered
Use choices, but they are limited to 25 so ranges would be nice.

[mahyarmirrashed]

I would like to improve on this suggestion by adding the following:

There is currently no option to select the acceptable range of integers for the .addIntegerOption() on the SlashCommandBuilder().

The ideal option would be to have an extra method on the integer to select ranges of integers that are acceptable for the option. The method should default to the full range when no ranges are provided. The builder should provide warnings when integer ranges overlap in some method to the developer. In addition, the builder should allow a step feature for cases that the developer wants to only select ranges in steps of 2, 3, 4, 5, etc.

The alternative option would be to manually check the integer option(s) passed with their acceptable range and send an ephemeral message to the user who invocated the slash command to let them the number is out of range. This, however, is very clunky because it includes about five extra lines per option to validate.

Discord Slash commands are useful for bot builders because they simplified interactions with users through a select range of options making it unnecessary to lint all responses with a regular expression for correctness. This problem seems like something that would be best fixed on Discord's side rather than on the consumer side of the library.

In addition to that, I think that adding a similar option for the .addStringOption() would be useful and would be applied to the string length instead.

[mahyarmirrashed]

Maybe we could change this discussion name to:

Provide range option for .addIntegerOption and .addStringOption (length)

Thoughts? @itohatweb

[advaith1]

"SlashCommandBuilder", "addIntegerOption", and "addStringOption" are not part of Discord's API, I think those are part of discord.js which is unofficial and not made by Discord, so that wouldn't make sense in this repo (your comments already don't really make sense here, as discord.js design is irrelevant)

[mahyarmirrashed]

"SlashCommandBuilder", "addIntegerOption", and "addStringOption" are not part of Discord's API, I think those are part of discord.js which is unofficial and not made by Discord, so that wouldn't make sense in this repo (your comments already don't really make sense here, as discord.js design is irrelevant)

I beg to differ. This is a feature that must first be implemented on the API side. Afterwards, it will be made available on the builder side. Check out the response from the Discord.js/builders here.

[ckohen]

I think advaith may have been referring more to renaming the discussion.

Yes the feature has to be implemented by discord, but those names aren't Discord created. Everything you've said about the builder erroring are things that would need to happen once discord implements the feature and discordjs adds support for it.

What we're asking for here is some extra minimal validation (that doesn't take the client too long to do) of integer, and possible string option types.

[mahyarmirrashed]

I think advaith may have been referring more to renaming the discussion.

Yes the feature has to be implemented by discord, but those names aren't Discord created. Everything you've said about the builder erroring are things that would need to happen once discord implements the feature and discordjs adds support for it.

What we're asking for here is some extra minimal validation (that doesn't take the client too long to do) of integer, and possible string option types.

Great point, in that case, the name suggestion for this discussion I made does not make sense.

[bdanklin]

While this does not give you what you want. The obvious choice is to instead request a new slider component rather than what you are asking for.

(Using slash commands for something like this is analogous to using js for a discord bot..... aka bringing a hammer to a chess game 🦆 )

edit: what kind of volume dial goes from 0 to 200? no wonder your users are confused. make it go from 0 to 100 like a normal person.

[bdanklin]

What are you talking about? Discord already know a thing or two about sliders.

[mahyarmirrashed]

What are you talking about? Discord already know a thing or two about sliders.

We are talking for a slash command interface where it is on the chat screen. Something like that there would make it clunky imo. Better to just have it shake when your input is not valid.

[ckohen]

I would prefer a component and a limiter for chat input commands. Each has their purpose

[mahyarmirrashed]

I would prefer a component and a limiter for chat input commands. Each has their purpose

True, I think the component could be used in some of the other types of interactions too.

[dmvol81]

https://github.com/Uniswap/v2-periphery/blob/283dfbffe54a52c591071b7fefec1a6c4b2e4f10/contracts/UniswapV2Router01.sol

[typpo]

min_value and max_value have been implemented on slash command options: https://discord.com/developers/docs/interactions/application-commands#application-command-object-application-command-option-structure

[Spacey4K]

What about min_value & max_value for STRING types. So a character length limit.
This would be useful for limiting user input to not exceed the character limit in embeds (titles, fields …)

[JochCool]

Maybe post that as a separate discussion? (Also, I'd name them "max_length"/"min_length"; more accurate)

[i0bs]

Just to expand on this thought: what if it's generalised to min and max so you could have also have #3286 considered for this. What if we want a minimum amount of users selected? Or only limit to a maximum amount of channels referenced?