-
Notifications
You must be signed in to change notification settings - Fork 176
Support all the methods from the API #66
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
Comments
Yes. Initially, that module was designed for "sending message" purpose. I was convinced, that adding a new simple wrapper for every API call is redundant. Could you show a code of a method that defined by a configuration file, in order to avoid misunderstandings :) |
yml file is a subset of JSON, it's just define a json file like that: {
"methods": [
{
"endpoint": "channels:history",
"type": "GET",
"parameters": {
"name": "channel"
}
}
]
} and the parse it and assign to the get/post private method from the class. Sounds good? |
and aswell, we can have the documentation dynamic, just adding jsDoc it will output that on the README. Like swagger. |
Yes, sounds good. Anyway, I don't see a huge advantage over js functions. But... could you do PR? :) |
This feature would be really nice to have but i also don't really see why we shouldn't just use JS functions. |
Are we still supporting all methods from the Slack API? |
I'm closing this since there's no activity related to that. |
Hey @mishk0,
Seems that this module doesn't support all the endpoints available on the API api.slack.com/methods.
What about have some methods defined by a configuration file (often a yaml) that defines all the methods from the API with the parameters and have other "custom" methods implemented directly in JavaScript. Like getGroupById, getChannelById, etc...
We could apart of support all the methods from the API, explain how this slack-api-client add extra value of the raw slack API.
Thanks :D
The text was updated successfully, but these errors were encountered: