-
Notifications
You must be signed in to change notification settings - Fork 20
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
treehouses message discord [servers|channels|read|send]
new subcommands (fixes #2152)
#2151
base: master
Are you sure you want to change the base?
Conversation
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All new features work. I left comments on possible improvements.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to add new subcommands to:
_treehouses
README.md
help.sh
message.sh
(message_help)
|
|
I will need to check up on the documentation to see the limitations on how far back the message history goes. |
I'm holding off on this until @dogi decides if he wants to use the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All features work when tested. I made some optional suggestions. Waiting on @dogi to decide whether to use the user token (this implementation) or use the Oauth/Bot for the API token.
treehouses message discord [servers|channels|read|send]
new subcommandstreehouses message discord [servers|channels|read|send]
new subcommands (fixes ##2152)
treehouses message discord [servers|channels|read|send]
new subcommands (fixes ##2152)treehouses message discord [servers|channels|read|send]
new subcommands (fixes #2152)
@JLKwong looked at the documentation and I believe it defaults to 50 messages. |
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left recommendations based on testing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will create PR's for my suggestions after @dogi merges this PR.
@rjpadilla It seems that the Discord authorization token expires after a while. I've been running |
New
treehouses message discord
featuresFour new subcommands for
treehouse message discord
:servers
- list all servers the user is inchannels
- list all channels in the chosen server by the userread
- read all messages in the chosen server and channelsend
- send a message in the chosen server and channelHow to use
In the Discord Web Client, type
ctrl+shift+i
to pull up the browser's Developer Tools.Head to the Network tab and filter the url with
/api
(you may need to reload the page).Click on any of the requests and in the headers windows on the right, copy the Authorization's value.
This value will be your API token.
Add your apitoken with the command
treehouses message discord authorize <apitoken>