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

Make !source a slash command #2424

Open
DMFriends opened this issue Mar 1, 2023 · 5 comments
Open

Make !source a slash command #2424

DMFriends opened this issue Mar 1, 2023 · 5 comments
Labels
a: frontend Related to output and formatting a: information Related to information commands: (doc, help, information, reddit, site, tags) p: 3 - low Low Priority status: approved The issue has received a core developer's approval t: enhancement Changes or improvements to existing features

Comments

@DMFriends
Copy link

DMFriends commented Mar 1, 2023

I think the !source command should be made a slash command because, similarly to the reason why we made !tags a slash command, it would make it easier to look up the name of a cog without having to leave the Discord channel if you don't exactly remember the name offhand.
Note: I'm fine with someone else implementing this if approved, I don't really have the time right now to contribute at all

@Robin5605
Copy link
Contributor

I assume we'd also implement autocomplete here?

@DMFriends
Copy link
Author

DMFriends commented Mar 2, 2023

Yes, that's what I was referring to when I said "it would make it easier to look up the name of a cog", just didn't have the word "autocomplete" offhand when I wrote the issue.

@mbaruh
Copy link
Member

mbaruh commented Mar 4, 2023

Sounds fine to me, although the cog would need to go through some structural changes to support auto-complete. Technically it could produce a static mapping of all possible terms and their appropriate info (type, link etc.), since I don't see why it would change without the bot restarting in the process. It would have to be the last cog to be loaded though, or there would have to be some other mechanic that would allow the mapping to fill up.

@mbaruh mbaruh added a: frontend Related to output and formatting p: 3 - low Low Priority a: information Related to information commands: (doc, help, information, reddit, site, tags) t: enhancement Changes or improvements to existing features status: approved The issue has received a core developer's approval labels Mar 4, 2023
@Ibrahim2750mi
Copy link
Contributor

Can I be assigned this?

@mbaruh
Copy link
Member

mbaruh commented Mar 5, 2023

Assigned

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a: frontend Related to output and formatting a: information Related to information commands: (doc, help, information, reddit, site, tags) p: 3 - low Low Priority status: approved The issue has received a core developer's approval t: enhancement Changes or improvements to existing features
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants