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

add include_all_metadata to conversations.replies API #1139

Conversation

ebkn
Copy link
Contributor

@ebkn ebkn commented Nov 30, 2022

WHAT

add include_all_metadata param to conversations.replies API
doc: https://api.slack.com/methods/conversations.replies#arg_include_all_metadata

WHY

Metadata is supported (Thanks #1083), but does not support conversations.replies API. include_all_metadata param is needed to get metadata.

@kanata2 kanata2 self-requested a review December 2, 2022 18:35
Copy link
Member

@kanata2 kanata2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@kanata2 kanata2 merged commit 6c7bfc2 into slack-go:master Dec 7, 2022
@ebkn ebkn deleted the feature/include-metadata-on-conversations-replies-api branch December 8, 2022 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants