-
Notifications
You must be signed in to change notification settings - Fork 214
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
Support Oscar 0.6 #34
Conversation
Related to #32.
- Only check for shipping address when basket indicates that one would be required.
Changed in Oscar 0.6.
I see an other poroblem You can replace
Then, however, i have this error: |
Conflicts: paypal/express/gateway.py paypal/express/views.py
- Pass basket to get_shipping_address - Pass basket to get_shipping_method - Ensure strategies are assigned to reloaded frozen baskets - Use a build_submission method - Pass both values to FixedPrice shipping method - Use renamed bankcard attributes in tests
The folders are required so the image_not_found image can be automatically created.
I've fixed the issues arising from changing APIs in Oscar 0.6. Tests are passing and everything seems to be working. I still need to test some US scenarios where tax isn't known in advance. |
except Basket.DoesNotExist: | ||
messages.error( | ||
self.request, | ||
_("No basket was found that corresponds to your " | ||
"PayPal transaction")) | ||
return HttpResponseRedirect(reverse('basket:summary')) | ||
else: | ||
# Assign strategy to basket instance | ||
basket.strategy = Selector().strategy(self.request) |
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.
Oscar 0.4 & 0.5 don't have selector class
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.
I think you need it to determine any basket line prices or totals.
We'll probably need to stop supporting 0.4 and 0.5 for the next release.
This a WIP branch to track progress on supporting Oscar 0.6. It will be finished off and released when Oscar 0.6 is released.
oscar_testsupport