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.

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

Add deploy task for payment reminder email: 169468997 #2895

wants to merge 5 commits into
base: master


Copy link

lynzt commented Nov 1, 2019


Add step in circle config to deploy new subtask.

Reviewer Notes

Anything else I should add to not bring down master?

Code Review Verification Steps

  • Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
  • User facing changes have been reviewed by design.


@lynzt lynzt requested review from chrisgilmerproj, mr337 and Ryan-Koch Nov 1, 2019
@lynzt lynzt added the ttv label Nov 1, 2019
Copy link

chrisgilmerproj left a comment

Let's deploy this to experimental and try it out! I think it ought to work. I can also update the cron schedule if you have a test plan. Then we can see it working in the cloudwatch logs.

@lynzt lynzt force-pushed the lt-add-deploy-task-169468997 branch from c9a7b0c to be9cbaa Nov 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
2 participants
You can’t perform that action at this time.