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.

Sign up for GitHub

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

Jump to bottom

Is it possible to set reply_broadcast in say() function? #738

Closed
jacklowrie opened this issue Oct 14, 2022 · 2 comments · Fixed by #741
Closed

Is it possible to set reply_broadcast in say() function? #738

jacklowrie opened this issue Oct 14, 2022 · 2 comments · Fixed by #741
Assignees
@seratch
Labels
area:async area:sync improvement question Further information is requested
Milestone
1.15.2

Comments

@jacklowrie
Copy link

Looking in the api docs at the Say class, it seems like this might not be supported, since reply_broadcastisn't a possible parameter, but wanted to check in case I missed something.

If it's not, what would be a better way to send a broadcast reply message from a listener function?

Thanks!
@jacklowrie jacklowrie mentioned this issue Oct 14, 2022
Closed
@seratch seratch added the question Further information is requested label Oct 14, 2022
@seratch
Copy link
Member

seratch commented Oct 14, 2022

Hi @jacklowrie, thanks for asking this! Indeed, reply_broadcast is not in the keyword arguments but passing it actually works: https://github.com/slackapi/bolt-python/blob/v1.15.1/slack_bolt/context/say/say.py#L47 We will update the code for more clarity.

@seratch seratch self-assigned this Oct 14, 2022
@seratch seratch added this to the 1.15.2 milestone Oct 14, 2022
@jacklowrie
Copy link
Author

Thanks so much! works like a charm.

seratch added a commit to seratch/bolt-python that referenced this issue Oct 18, 2022
@seratch
Fix slackapi#738 Add more keyword args to say utility
42c05c4
@seratch seratch mentioned this issue Oct 18, 2022
Merged
8 tasks
seratch added a commit that referenced this issue Oct 18, 2022
@seratch
Fix #738 Add more keyword args to say utility (#741)
31146a2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@seratch seratch

Labels
area:async area:sync improvement question Further information is requested
Projects
None yet
Milestone
1.15.2
Development

Successfully merging a pull request may close this issue.

Fix #738 Add more keyword args to say utility seratch/bolt-python
2 participants
@seratch @jacklowrie