-
Notifications
You must be signed in to change notification settings - Fork 714
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
Fix method_complexity issue in sendgrid/helpers/mail/ganalytics.py Fixes #498 #514
Conversation
We can’t merge your PR without the CLA being signed. [ ] Sign the CLA before November 3rd Then we will comment with a link to get your shirt! |
@mbernier @thinkingserious |
Hi @chetankm-cs, It looks like your commit was made by another user account "Chetan". Maybe the easiest thing to do is to re-create the PR under your @chetankm-cs account, reference this issue and close it. With Best Regards, Elmer |
@thinkingserious I have changed the author, the CLA check has passed. Thanks. |
Thank you @chetankm-cs! Please click here to fill out the form. Thanks! |
Codecov Report
@@ Coverage Diff @@
## master #514 +/- ##
==========================================
+ Coverage 87.91% 88.36% +0.45%
==========================================
Files 30 30
Lines 935 937 +2
Branches 115 112 -3
==========================================
+ Hits 822 828 +6
Misses 55 55
+ Partials 58 54 -4
Continue to review full report at Codecov.
|
Hello @chetankm-cs, |
Fixes
Fixes #498
Checklist
Short description of what this PR does:
If you have questions, please send an email to Sendgrid, or file a Github Issue in this repository.