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
feat(functions/slack): add GCF v2 decorators #7889
Conversation
@@ -89,6 +90,7 @@ def make_search_request(query): | |||
|
|||
|
|||
# [START functions_slack_search] | |||
@functions_framework.http |
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.
Sorry I'm not too informed about this decorator, is this for good practice? If this is a recommended approach, and this is going to show up on the Sample Browser as well, I think it might be good to be able to point users towards some sort of documentation that explains why using this might be beneficial!
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 don't think it was documented; This PR GoogleCloudPlatform/functions-framework-nodejs#347 is the most detailed bit I could find. Thoughts on whether we should point to it?
Co-authored-by: Dan Lee <71398022+dandhlee@users.noreply.github.com>
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.
* feat(functions/slack): add GCF v2 decorators * Fix lint * Fix copyright * Bots hate commas * Humans hate commas too * Fix copyright Co-authored-by: Dan Lee <71398022+dandhlee@users.noreply.github.com> Co-authored-by: Dan Lee <71398022+dandhlee@users.noreply.github.com>
✅ I've confirmed this works on GCF itself.