Skip to content

Hand over the matched option value into the autocomplete callback#1416

Open
codemasher wants to merge 3 commits into
discord-php:masterfrom
codemasher:handover-autocomplete-options
Open

Hand over the matched option value into the autocomplete callback#1416
codemasher wants to merge 3 commits into
discord-php:masterfrom
codemasher:handover-autocomplete-options

Conversation

@codemasher
Copy link
Copy Markdown
Contributor

@codemasher codemasher commented Oct 6, 2025

This saves digging through the entire Interaction instance again in order to find the focused value that triggered the callback.

@codemasher
Copy link
Copy Markdown
Contributor Author

I'm taking out the thoughts from the OP and leave them here, this is material for another day as there are currently too many open questions re this:

I'd also like to hand over the matched command/subcommand/subcommandgroup, for which I'd like to dissolve that recursive call (recursions are a bit overused in your library anyway for my taste :D). From what I gather, the options instance is nested similar to the description here, or is there something I'm overlooking?
3 nested foreach calls might look a bit clunky, but are probably faster and leave the option to hand over the command names too.

re OP: LGTM?

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.

1 participant