Skip to content
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

Release 1.0.2 with some updates (see the changelog) #62

Merged
merged 7 commits into from
Mar 1, 2016

Conversation

maerteijn
Copy link
Member

No description provided.

@maerteijn
Copy link
Member Author

Also fixes django-oscar/django-oscar#2000 with the current release of django-oscar

@maerteijn maerteijn force-pushed the releases/1.0.2 branch 3 times, most recently from 992b485 to a3c4736 Compare March 1, 2016 19:45
@specialunderwear
Copy link
Member

@keeshink ik ga hem mergen

response = session.post('http://localhost:8000/api/basket/add-product/', json=data)
lines_url = response.json()['lines']
response = session.get(lines_url)
print(response.content)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

print?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the example in the docs so yes, print :)

So you can do post actions like sending a confirmation email or
redirect to the payment provider if you like without overriding the
CheckoutView
@maerteijn maerteijn force-pushed the releases/1.0.2 branch 2 times, most recently from 14a1438 to f69796e Compare March 1, 2016 21:59
specialunderwear added a commit that referenced this pull request Mar 1, 2016
Release 1.0.2 with some updates (see the changelog)
@specialunderwear specialunderwear merged commit bdf48b6 into master Mar 1, 2016
@specialunderwear specialunderwear deleted the releases/1.0.2 branch March 1, 2016 22:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants