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

Optimize tag command #439

Closed
Kaktushose opened this issue Jul 18, 2023 · 3 comments · Fixed by #441
Closed

Optimize tag command #439

Kaktushose opened this issue Jul 18, 2023 · 3 comments · Fixed by #441
Assignees
Labels
enhancement New feature or request

Comments

@Kaktushose
Copy link

Kaktushose commented Jul 18, 2023

Type

Enhancement of an existing feature

Description

Problem

All /tag commands (except for /tag list) require the tag name as an option. But remembering all tag names is very hard. This results in:

  • invalid command executions
  • excessive use of /tag list
  • and ultimately avoidance of using tags at all

Solutions

This issue could be resolved in three different ways

a) using choices

The name option could use choices instead.

Pros:

  • the user can directly see all the tags that are available

Cons:

  • you can only add up to 25 choices
  • all tag commands need to be updated everytime a tag gets added/deleted/modified

b) using autocomplete

autocomplete allows the bot "[...] to dynamically return option suggestions to a user as they type".

Pros:

  • easy to implement
  • allows unlimited amount of choices

Cons:

  • user still has to remember the tags to a certain extend

c) using both choices and autocomplete

This approach combines the both solutions discussed prior. The bot would use choices for up to 25 tags. After that, autocomplete would be used instead. This is by far the most complex solution, but it would also provide the best user experience.

Conclusion

From my perspective both as a user and a developer I'd prefer solution b) since tags are more of a niche feature and the cost-benefit factor does not pay off with the other solutions

@zekroTJA
Copy link
Owner

Requires zekroTJA/ken#18.

@zekroTJA
Copy link
Owner

zekroTJA commented Jul 19, 2023

In the same move, the list command for tags should be made ephemeral.

@zekroTJA
Copy link
Owner

zekroTJA commented Jul 21, 2023

I've decided on your option B). Command auto completion has now been implemented in Ken and also beein implemented for the tag command. When no input is presented, the full list of tags (capped to 30 at max) is shown to the user, so they don't even need to remember most of the tags at all.

Thank you very much for your feedback! :)

@zekroTJA zekroTJA mentioned this issue Jul 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants