-
Notifications
You must be signed in to change notification settings - Fork 235
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
Conditional Lazy Listener? #482
Comments
Hi @SigniantJoe, thanks for writing in! Unfortunately, there is no way to run lazy listeners based on the ack listener behavior. the current recommendation for your use case at this moment is to have the same validation in the beginning of the lazy listeners. With that being said, we may consider adding a new configuration / mechanism to support this use case in a better way. |
@seratch thank you for confirming the behaviour. I do think this would be a great enhancement, especially as sometimes we need to query external services or do relatively heavy validation |
Is there a way to make it so that the lazy function triggers conditionally based on the ack response?
For example, I want my ack to do parameter validation and potentially reject the request. Right now, my solution is to have the lazy function run the same parameter validation as the ack function, but it seems a bit wasteful. I'd much rather the lazy function not get triggered at all in the scenario where the ack function returns errors to the user.
Reproducible in:
The
slack_bolt
versionslack-bolt==1.9.1
Python runtime version
Python 3.8.12
OS info
Steps to reproduce:
Run something like this:
Expected result:
Don't do the thing if validation fails.
Actual result:
Does the thing even if validation fails.
Requirements
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.
The text was updated successfully, but these errors were encountered: