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.

Sign up for GitHub

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

Jump to bottom

Log a warning when we fail to send a notification to the gov.notify a… #47

Open

cofaulco wants to merge 2 commits into master from Failure_to_send_logging

Contributor

cofaulco commented Dec 4, 2019

…pi. Need to check how this code is handling retries


          Log a warning when we fail to send a notification to the gov.notify a…

cbe5f4d

…pi. Need to check how this code is handling retries

cofaulco requested a review from dashton82

December 4, 2019 13:40


          Better context in log message

ac52f70

dashton82 approved these changes

View reviewed changes

Contributor

EwanNoble commented Dec 13, 2019

Would this not be more suitable at Error level?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet