Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

open source push 3-11-2021 #29

Merged
merged 1 commit into from
Mar 22, 2021
Merged

open source push 3-11-2021 #29

merged 1 commit into from
Mar 22, 2021

Conversation

noxasaxon
Copy link
Contributor

@noxasaxon noxasaxon commented Mar 12, 2021

Changes:

  • list of Slack Oauth Scope permissions granted to the bot in the README
  • add a DynamoDB table for caching usernames. you can choose to disable this feature by commenting out the CACHE_USERS_TABLE environment variable in serverless.yml and the DynamoDB file in serverless.yml
  • add update_cached_users lambda function for automatically populating the username table
  • add an "integration test" playbook that chains together all the lambdas intended for use in playbooks (not the endpoints).
  • add an integration test "test input" that is uploaded to SSM. This file's contents can be used to trigger the slack integration test playbook via the create_events function in the main socless stack. NOTE: you will need to modify this file's event details for your Slack environment
  • reduce duplicate code among all functions by updating the ./common_files/slack_helpers.py file with generic functions and improved error handlng

Contributing to Twilio

All third-party contributors acknowledge that any contributions they provide will be made under the same open-source license that the open-source project is provided under.

  • I acknowledge that all my contributions will be made under the project's license.

@noxasaxon noxasaxon requested a review from ubaniabalogun March 15, 2021 19:01
@noxasaxon noxasaxon merged commit a931da6 into master Mar 22, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants