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

PayOnDelivery backend doesnt work #24

Closed
bmihelac opened this issue Apr 8, 2011 · 5 comments
Closed

PayOnDelivery backend doesnt work #24

bmihelac opened this issue Apr 8, 2011 · 5 comments

Comments

@bmihelac
Copy link
Contributor

bmihelac commented Apr 8, 2011

it calls a bunch of methods that does not exists anymore.

should it call confirm_payment and return render success template?

@chrisglass
Copy link
Contributor

Yes, it should be refactored, indeed.
Ping me here if you're up for it, otherwise I'll do it ASAP

@bmihelac
Copy link
Contributor Author

bmihelac commented Apr 8, 2011

looking at paypal/postfinance, it returns:
but i feel it needs something more... so you can do it

 def simple_view(self, request):
     return HttpResponse()

@chrisglass
Copy link
Contributor

Should be fixed now, even though it's still not the most interesting response ever (it now uses shop.success())

@bmihelac bmihelac reopened this Apr 10, 2011
@bmihelac
Copy link
Contributor Author

there is one typo, i am sending pull request

@bmihelac
Copy link
Contributor Author

i am closing this

jrief added a commit that referenced this issue Jan 29, 2016
Fix pip double requirements problem
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

No branches or pull requests

2 participants