-
|
I'm using dj-stripe webhook-handlers to perform different operations on webhooks (e.g. sending mail to customer when subscription is updated). The event-data contains e.g. all subscription-data event-specific data. But all event-data is a simple dict. How do i get the dj-stripe object from/inside the event handler? Can the dict be converted into a djstripe-object? Is this how it is supposed to work? (Seems strange to perform another api-call for data that is already present in the web-handler) |
Beta Was this translation helpful? Give feedback.
Replies: 2 comments 5 replies
-
|
Why sync the data from the stripe api if the webhook just received the latest data Heres my suggestion: from django.db import transaction
from djstripe import webhooks
from djstripe.models import Subscription
def do_something(subscription_id):
subscription = Subscription.objects.get(stripe_id=subscription_id)
# Do something with the subscription
@webhooks.handler("checkout.session.completed")
def checkout_session_completed(event, **kwargs):
session_data: Dict[str, Any] = event.data["object"]
subscription_id: Optional[str] = session_data["subscription"]
if subscription_id:
transaction.on_commit(do_something, subscription_id)(I did not test teh above code if it works) |
Beta Was this translation helpful? Give feedback.
-
|
So the Dj-stripe object in the database is already the newest updated object (with the current event-data)? That means that djstripe updates the database object based on the webhook-event-data before the webhook-handler (shown above) is executed. It that correct? If that is correct - why dosen't the webhook.handler provide direct access to the dj-stripe object? Wouldn't it be very usefull e.g. if event.data[object] would contain a dj-stripe-object and not a dict? |
Beta Was this translation helpful? Give feedback.
-
|
I want to clarify the terms we are using Full disclosure I am relativly new to this project so I may be wrong. IMO the djstripe signal should include the local Django object (which was just updated) (its my understanding that in v3.0 the signal thing will be different but I don't have clarity |
Beta Was this translation helpful? Give feedback.
-
|
Thank you very much for the clarification!!! |
Beta Was this translation helpful? Give feedback.
-
|
Maybe you can file a feature request to include the django object in the signal. |
Beta Was this translation helpful? Give feedback.
-
|
Feel free to open a PR |
Beta Was this translation helpful? Give feedback.
-
|
I propose a solution in #2088 |
Beta Was this translation helpful? Give feedback.
Why sync the data from the stripe api if the webhook just received the latest data
Heres my suggestion:
(I did not test teh above code if i…