-
Notifications
You must be signed in to change notification settings - Fork 285
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
CustomerUpdatedWebhook: use provided customer object from the event #371
CustomerUpdatedWebhook: use provided customer object from the event #371
Conversation
pinax/stripe/webhooks.py
Outdated
customers.sync_customer(self.event.customer) | ||
cu = None | ||
try: | ||
cu = self.event.message['data']['object'] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
double-quotes please :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure. Anything apart from that?
Shouldn't there be a test (change)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also: maybe we could have pylint or something similar to report this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let machines work for us.. #372
f649448
to
04f9f9a
Compare
Updated branch, so please squash-merge. |
33878a9
to
18403a9
Compare
@paltman |
18403a9
to
efa780d
Compare
Codecov Report
@@ Coverage Diff @@
## master #371 +/- ##
==========================================
+ Coverage 99.84% 99.84% +<.01%
==========================================
Files 31 31
Lines 1297 1302 +5
Branches 98 98
==========================================
+ Hits 1295 1300 +5
Partials 2 2
Continue to review full report at Codecov.
|
It saves a network roundtrip, and might even be necessary if events are supposed to be processed in order.
TODO: