Adding Paypal Support

[HarryLafranc]

Changelog Link
https://stripe.com/docs/changelog#may-9,-2023

Describe the Change
Paypal was just introduced in Stripe, it would be nice to implement the bare minimum to make it work. I'm mainly getting errors in django admin because _human_enum_values() don't recognize "paypal" value. I don't know dj-stripe enough internally to know if others things must be reworked, but I will try to take a look at it!

[colmmcmullan]

+1
It would be great to support PayPal.

I use Stripe Checkout and was able to hack a working test simply by adding the following to PaymentMethodType in djstripe/enums.py:

paypal = _("PayPal")

I assume that for other workflows something would have to be added to payment_methods.py though?

[arnav13081994]

@HarryLafranc @colmmcmullan Yes the only change that is required is addition of paypal = _("PayPal") to the PaymentMethodType enum. Unfortunately this fix won't be in 2.* releases as we are working on revamping the entire dj-stripe repo.

[agusmakmun]

Any updates for adding paypal field in class PaymentMethod(StripeModel):?

[jleclanche]

@agusmakmun it's live in 2.8.3

[agusmakmun]

hello @jleclanche thank you for quick answer.

We didn't see any paypal field in the class PaymentMethod(StripeModel) model.

https://github.com/dj-stripe/dj-stripe/blob/2.8.3/djstripe/models/payment_methods.py#L1021

only seeing the card, paynow and some related.

as paypal already have their own field:
https://docs.stripe.com/api/payment_methods/object#payment_method_object-paypal

[jleclanche]

Ah, I see what you mean. Okay yeah that will not be in 2.8. In 3.0 the class will change to always be up to date via a stripe_data field.

[agusmakmun]

@jleclanche I see. What we should do for this migration? Of course will have breaking changes right?

[jleclanche]

Unfortunately, yes, it'll be quite a significant breaking change.

[agusmakmun]

Could you please give us the documentation how to smoothly migrating the dj-stripe from 2.x version to 3.x?
It will helps us lot, especially for old members that already using this package for long time.

Thank you btw 🙏 👍

[jleclanche]

@agusmakmun absolutely. Unfortunately the migration path doesn't exist at this time; I will do a write-up when we get to the release date.