-
Notifications
You must be signed in to change notification settings - Fork 248
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
Updating lazy lambda OAuth example #444
Conversation
|
||
sys.path.insert(1, "vendor") |
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.
It doesn't look like this works. AWS Chalice seems to include vendor/
automatically but AWS Lambda not so much. I can't find more information on this, but I do remember before this did work! But, it has been probably 2 years or so since I've used Python in Lambda, so perhaps that behaviour has changed.
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.
Good catch 👍
examples/aws_lambda/README.md
Outdated
- `SLACK_INSTALLATION_S3_BUCKET_NAME`: The name of one of the S3 buckets you | ||
created | ||
- `SLACK_STATE_S3_BUCKET_NAME`: The name of the other S3 bucket you created | ||
- `SLACK_LAMBDA_PATH`: ??? TODO |
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.
@seratch what is this env var used for, do you know?
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.
This is no longer needed. You can safely delete it!
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.
Thanks for updating the example 👍
examples/aws_lambda/README.md
Outdated
- `SLACK_INSTALLATION_S3_BUCKET_NAME`: The name of one of the S3 buckets you | ||
created | ||
- `SLACK_STATE_S3_BUCKET_NAME`: The name of the other S3 bucket you created | ||
- `SLACK_LAMBDA_PATH`: ??? TODO |
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.
This is no longer needed. You can safely delete it!
|
||
sys.path.insert(1, "vendor") |
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.
Good catch 👍
Codecov Report
@@ Coverage Diff @@
## main #444 +/- ##
=======================================
Coverage 91.34% 91.34%
=======================================
Files 167 167
Lines 5499 5499
=======================================
Hits 5023 5023
Misses 476 476 Continue to review full report at Codecov.
|
Updating the lazy lambda OAuth example with instructions
Category (place an
x
in each of the[ ]
)slack_bolt.App
and/or its core componentsslack_bolt.async_app.AsyncApp
and/or its core componentsslack_bolt.adapter
/docs
Requirements (place an
x
in each[ ]
)Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.
./scripts/install_all_and_run_tests.sh
after making the changes.