Comments (4)
Based on what you've said, it looks like we're going to have to be compatible with all the different things they might be sending. If you can post examples of different things you are seeing that would really help. A patch that fixes the issues would also be a huge help. Thanks so much!
from django-paypal.
I've been running this branch for the last week or so without any issues, but it's a bit hacky to push upstream:
master...djw:return-query-changes
Given that payment_date
is now arriving in ISO format in the query parameters, but still in PayPalDateTimeField
format in the postback response, I think the options are:
- Extend
PayPalDateTimeField
to support ISO format (perhaps requiring a dependency on python-dateutil) - Use separate form classes to handle query parameters and the postback response, and probably ignore all the new parameters, including the ISO format date.
I lean towards the second option, but would like your opinion before working on a patch. :)
from django-paypal.
Your second option sounds fine if you feel like working on a patch. Thanks!
from django-paypal.
Apologies for the delay, I've created a new branch which implements the fix following Option 2 above:
I'll test this out for a few days and then submit a PR.
from django-paypal.
Related Issues (20)
- Upgrade or Downgrade subscription plan HOT 1
- Listing the payments HOT 1
- Paypal notify url webook expecting x-www-form-urlencoded but received application/json from paypal HOT 1
- Testing with paypal sandbox and ngrok resulting in invalid postback HOT 1
- Received 3 IPNs for the same transaction, only one flagged as duplicate HOT 3
- Paying directly with a debit card HOT 1
- Working with react js HOT 1
- version 2.0 - update docs and tag HOT 3
- Encrypted form gives different interface on PayPal HOT 1
- Custom data got truncated HOT 4
- "Things don't appear to be working at the moment. Please try again later." HOT 3
- Add pay now button
- untested warning for is_subscription HOT 4
- How to use this sdk in django-rest-framework HOT 2
- Any pointers on how to integrate with Django Oscar?
- application/json vs application/x-www-form-urlencoded
- Increase the size of the logo HOT 2
- Add `Django 5.0` compatibility. HOT 2
- Outdated package and documentation HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from django-paypal.