Vue-storefront: Shouldn't we emplement payment-requests spec?

[pkarw]

https://www.w3.org/TR/payment-request/

[pkarw]

https://developers.google.com/web/fundamentals/payments/deep-dive-into-payment-request#paymentrequest_shim

[pkarw]

https://stripe.com/docs/payment-request-api

[MikeSheward]

I think we should implement this, MVP it with basic payment provider integration (stripe for example) and then move to fully support the Payment Request API with any further changes in a further milestone perhaps, including Apple Pay etc.

[pkarw]

Go @BlitZ-UK Go! :)

[filrak]

@BlitZ-UK maybe you can take care of this one?

[MikeSheward]

Sure! With Stripe integration it'll be built right in anyway (to some extent). But in a future milestone it would be good to add this support for manual Payment Request API integration.

_Edit: Actually i think it might be easy enough to get this up and running alongside the Stripe installation itself. Let's do this_

[pkarw]

@MikeSheward -> @2monkey is willing to help with this - as commented on #809

[pkarw]

@MikeSheward not sure, but this: https://github.com/DivanteLtd/vue-storefront/blob/master/doc/data/How%20to%20get%20Magento2%20order%20id%20after%20sync.md

May be interesting and helpfull. We're now storing the resulting Magento2 order id's in the vue-storefront-api redis cache:

https://github.com/DivanteLtd/vue-storefront-api/blob/0acfc471534c9f0abeaf5c263719e08118a96dcc/src/worker/order_to_magento2.js#L215

and

https://github.com/DivanteLtd/vue-storefront-api/blob/0acfc471534c9f0abeaf5c263719e08118a96dcc/src/api/sync.js#L16

[MikeSheward]

@pkarw thanks for them links, i'll take a look.

[2monkey]

@MikeSheward hey :) how far are you with this extension?

[MikeSheward]

Hey, it's currently very WIP. I've made some changes to the core to allow for adding dynamic payment provider sections to be added to the checkout, a few new events ect, which should make it easier for other payment providers to use too, i'll try and get something to show by the end of today.

[pkarw]

Great! Please send us some PR with the changes as I'm concerning how should we implement Paypal integration right now :)

[pkarw]

@MikeSheward maybe this new feature will be usefull for you guys: https://github.com/DivanteLtd/vue-storefront/blob/master/doc/components/Extending%20UI%20from%20extensions.md

I've added it with Paypal integration on mind :)

[MikeSheward]

Thanks, this is useful - To be uniform across extensions, I will refactor the Stripe to adhere to this doc instead, previously i was using the Dynamic Components method from the Vue docs, but this will also work just as well.

[pkarw]

@2monkey are on this?

[pkarw]

Regarding payments - please take a look at:

822 - any payment extension should be compliant with this "new stanard" - it will be merged in within few days but please treat it as already merged :)

and the: https://github.com/DivanteLtd/vue-storefront/blob/master/doc/components/Extending%20UI%20from%20extensions.md

[2monkey]

Okey, good to know.

[filrak]

It's ntoa ssigned to anyone so I will take care of this ;)