added discordjs docs command #1164
Conversation
derpyxd
added
type: feature
derpyxd
requested review from
pedrofracassi
and removed request for
pedrofracassi
src/commands/utility/docs.js
Outdated
| } | ||
|
|
||
| async run ({ channel }, query) { | ||
| const queryParams = new URLSearchParams({ src: 'stable', q: query }) |
There was a problem hiding this comment.
What if i dont want search in the stable branch?
There was a problem hiding this comment.
I'm going to hop in and suggest using a flag like --branch to choose a branch.
src/commands/utility/docs.js
Outdated
| module.exports = class Docs extends Command { | ||
| constructor (client) { | ||
| super({ | ||
| name: 'docs', |
There was a problem hiding this comment.
It should be djsdocs and aliases like djdocs
There was a problem hiding this comment.
Agreed. I'd say use "discordjs" as the main name and have a few derivatives from it, like "djsdocs", etc. Never just "docs", because we'll surely have docs commands for other things in the future.
You can use this |
I know, I was using the API so I wouldn't have so much work to handle the query to do the search on documentation |
|
By using that API localization is nearly impossible, so we better parse the original JSON ourselves. |
ok, i will remake this code and adaptating it to JSON documentation :) |
#Closes #508