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

emoji.list missing include_categories option #821

Closed
slushpupie opened this issue Aug 20, 2021 · 3 comments · Fixed by #1106
Closed

emoji.list missing include_categories option #821

slushpupie opened this issue Aug 20, 2021 · 3 comments · Fixed by #1106
Assignees
Labels
enhancement M-T: A feature request for new functionality project:slack-api-client project:slack-api-client
Milestone

Comments

@slushpupie
Copy link
Contributor

The Slack API for emoji.list has an option for include_categories, but the SDK does not support that option.

Note: As of 8/20, https://api.slack.com/methods/emoji.list does not mention this option, however, https://api.slack.com/methods/admin.emoji.list does, so I would consider this a public feature of the API. I have reported this documentation discrepancy to Slack directly, as well. So hopefully the documentation will be more clear soon.

@slushpupie slushpupie changed the title (Set a clear title describing the issue) emoji.list missing include_categories option Aug 20, 2021
@seratch seratch added enhancement M-T: A feature request for new functionality project:slack-api-client project:slack-api-client and removed untriaged labels Aug 20, 2021
@seratch seratch added this to the 1.12.0 milestone Aug 20, 2021
@seratch
Copy link
Member

seratch commented Aug 20, 2021

Hi @slushpupie, thanks for taking the time to flag this! Thanks for sharing the feedback with the team! Once the document is updated, we will add the parameter to the API client SDKs.

@seratch seratch modified the milestones: 1.12.0, 1.x Aug 27, 2021
@slushpupie
Copy link
Contributor Author

@seratch Any chance this will get updated? Do you have any sway with the API documentation team on getting this fixed there?

@seratch
Copy link
Member

seratch commented Jan 11, 2023

@slushpupie The server-side team made the option publicly available today. So I've added the option in this Java SDK and then just released a new patch version including the change. Thank you very much for your patience here. We're so sorry for being kept you waiting for such a long time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement M-T: A feature request for new functionality project:slack-api-client project:slack-api-client
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants