better_spree_paypal_express icon indicating copy to clipboard operation
better_spree_paypal_express copied to clipboard

API endpoints, Migrate PR #168 from 2-4-stable to 3-0-stable

Open givanse opened this issue 10 years ago • 7 comments
trafficstars

Migrate PR #168 from 2-4-stable to 3-0-stable.

I had 1 local failure, but I don't think is related to any of the PR changes. See: https://github.com/spree-contrib/better_spree_paypal_express/issues/176 Maybe Travis results will come up fine.

givanse avatar Aug 03 '15 03:08 givanse

Travis fails with the same error reported in https://github.com/spree-contrib/better_spree_paypal_express/issues/176, at least is consistent.

The failure is not related at all with the changes of this PR.

givanse avatar Aug 03 '15 04:08 givanse

@MisinformedDNA

williscool avatar Aug 12 '15 23:08 williscool

What's the latest on this @givanse ?

alexw23 avatar Aug 31 '15 21:08 alexw23

We should look at taking this logic for the API instead and putting it along with https://github.com/spree/spree_gateway/issues/232

alexw23 avatar Aug 31 '15 21:08 alexw23

The PR is complete, but its blocked by #176

givanse avatar Sep 01 '15 00:09 givanse

@givanse - I'm trying to use this code in my project. Can you point me to documentation on how the API requests / responses should look for this PR?

Something like this walkthrough would be great: https://github.com/spree-contrib/spree_api_examples/blob/master/examples/checkout/walkthrough.rb

lynndylanhurley avatar Apr 04 '16 20:04 lynndylanhurley

I never got this branch to work (see #176). I went with 2-4-stable.

When I was working with this gem I was building an Ember app, the addon I used for it is here: https://github.com/aethermx/spree-ember-paypal-express That repo should give you an idea of how the frontend is talking to the backend. It feels like this was ages ago. If that addon is of no help, feel free to ping me again.

givanse avatar Apr 04 '16 22:04 givanse