-
Notifications
You must be signed in to change notification settings - Fork 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
channels.info deprecated on Slack API #99
Comments
Hey yes shouldn't be too bad 🤞 hopefully no traps, I haven't checked it out you'll need to add it to the schema repo and use the codegen from this repo updating the sub repo. It gets a little messy but don't worry if it looks funky |
I also need this endpoint and had a look at the schema repo. However, I have no idea what I have to do to add it. |
I have been trying to look at the generator too, but without getting anything to work. |
I had a look at it and, well the scraper seems to be beyond repair. I created conversation.history manually, because that is the one I needed. We probably have to rewrite the scraper to use something like this Also, currently the slack crate only supportes get requests. So, we probably have to touch a lot... |
Hey I'm still here. If you want to do an PR to the schema repo that would be great. The scraper that worked off the docs was a good start but their docs also have the wrong types which means manual corrections. Last I checked slack's official spec also misses things. Now to be fair ours does too but at least there's something that can be done about that one. |
Slack has deprecated the channels.info endpoint (https://api.slack.com/methods/channels.info), and it no longer works at all with apps created after 10 June 2020. You're now supposed to use the conversations.info endpoint (https://api.slack.com/methods/conversations.info), but that doesn't appear to be supported by the crate.
I might take a stab at adding it; I assume it wouldn't be too complicated.
The text was updated successfully, but these errors were encountered: