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
fix!: Expose team_id
parameter for use with org-wide app
#1283
fix!: Expose team_id
parameter for use with org-wide app
#1283
Conversation
search.files search.messages
Hi @parsley42! I've noticed that you reviewed several PRs before, so tagging you here. This PR mainly goes through the list mentioned in the issue & adds |
Hi @candiduslynx - I'm retiring from being the de facto maintainer of this project; slack-go/slack needs maintainers, please email me at the address on my GitHub page if interested, thanks. |
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.
@parsley42 the tests have passed, so I'd like another set of eyes to merge this
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'd be interested in your thinking on updating the version, but otherwise this looks reasonable to me.
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.
LGTM
Fixes #1282