django-payments
django-payments copied to clipboard
Use JSONField for BasePayment.attrs
Need to wait until Django 2.2 is EOL.
Hi! I'm actually wanting to use this, since 2.2 has already been dropped, can I give you a hand with this?
I've rebased this, but it would be good to have some more testing before merging it.
I'll start using it right away, and let you know, my system processes a couple thousands purchases a day, so it's a fair trial.
Tests seem to still be failing on a bunch on combinations of Django + Python, so I'd hold testing it on a production environment for a bit.
Codecov Report
Attention: 13 lines in your changes are missing coverage. Please review.
Comparison is base (
84b52c9) 78.42% compared to head (2574ae3) 77.95%.
Additional details and impacted files
@@ Coverage Diff @@
## main #306 +/- ##
==========================================
- Coverage 78.42% 77.95% -0.48%
==========================================
Files 29 29
Lines 1979 1964 -15
Branches 244 244
==========================================
- Hits 1552 1531 -21
- Misses 310 317 +7
+ Partials 117 116 -1
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Looks like f4dfa5e5d80b1133accbdfccd83b52f7c1f271cb is breaking some tests, but it's not clear to me why.
Split the first commit into https://github.com/jazzband/django-payments/pull/385