Skip to content

Conversation

@heysagnik
Copy link

What does this PR do?

This PR fixes the error listed in the issue. Now the when notifyError is called it sends the required error logs to the desired email.

Test Plan

I haven't tested the code yet. but according to my knowledge this should fix the issue.

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work. Screenshots may also be helpful.)

Related PRs and Issues

Checklist

  • Have you read the Contributing Guidelines on issues?
  • If the PR includes a change to an API's metadata (desc, label, params, etc.), does it also include updated API specs and example docs?

@stnguyen90
Copy link
Contributor

Thanks, but this doesn't solve the problem. In addition, please make sure you're assigned to the issue before submitting a PR.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants