Migrating !tag to slash commands

[Canttuchdiz]

Migrating !tag to a slash command. Allowing for autocomplete, and a subcommand /tag help or /tags listing all tags. That list could also be ephemeral, so the user can see all the tags. Also, if possible, make the main command /tag, and the param would be autocomplete.

[JacobCoffee]

I believe this is a duplicate of #2362

[ChrisLovering]

I believe this is a duplicate of #2362

I suggested splitting out the issue in this way, see my recent comment in the issue you linked for context.

and a subcommand /tag help or /tags listing all tags. That list could also be ephemeral, so the user can see all the tags.

We could make the tag name argument optional, so if not given it lists all the tags in a paginated embed, like it does currently.

Also, if possible, make the main command /tag, and the param would be autocomplete.

I agree with this, we should use fuzzy matching for auto complete logic, similar to the current tag suggestion autocomplete, so that users can use the same short-hand names for tags as they currently do.

[Ibrahim2750mi]

can I be assigned this?

[wookie184]

I'm not sure if the intention of this issue is to replace the direct invocation of tags (e.g. through !code), or just the actual !tag command.

I think replacing the actual !tag command makes sense, but not removing the direct invocation. It does not seem like an improvement to go from !code to /tag get code, pressing enter, then having to press enter again after it autocompletes to /tag get codeblock, all with a bunch of discord UI popping up in the process. Having /tag with autocomplete would be useful for finding a tag if you're not sure if exists, but I think we should also keep the faster shorthand.

[Ibrahim2750mi]

I completely agree with what wookie said, I asked in the discord server but was advised it is a full migration.

Also @wookie184 your review comments makes sense now, sorry for misunderstanding.

If we all are on the same page for keeping the bot behaviour towards !code same, then I can implement it, although it would require a separate function get_command_ctx() in tags.py which will be invoked from try_get_tag_ctx() from error_handler.py

[Robin5605]

I'm also in favor of what wookie said. Seems like a downgrade to remove to !tag functionality. Ideally the slash command would increase visibility

[mbaruh]

Yeah let's keep the direct usage for now.