-
Notifications
You must be signed in to change notification settings - Fork 30
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: Send email to admins on failed payment #947
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅ ✅ All tests successful. No failed tests found. 📢 Thoughts on this report? Let us know! |
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. Additional details and impacted files@@ Coverage Diff @@
## main #947 +/- ##
=======================================
Coverage 96.24% 96.25%
=======================================
Files 826 826
Lines 19020 19035 +15
=======================================
+ Hits 18306 18322 +16
+ Misses 714 713 -1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
09da596
to
3e28326
Compare
aebf04f
to
f81dfdc
Compare
This reverts commit 97f26ab.
This PR adds code to send emails (through the send email worker task) to all owner admins when an invoice payment fails. We are doing this because the current stripe behavior only sends the notification email to the address associated with the stripe customer.
Depends on codecov/worker#835
Closes codecov/engineering-team#2483
Closes codecov/engineering-team#2484
Closes codecov/engineering-team#2485