stripe_official icon indicating copy to clipboard operation
stripe_official copied to clipboard

sets `stripe_fullname` and `stripe_email` in the payment form card te…

Open jagui opened this issue 5 years ago • 11 comments

sets stripe_fullname and stripe_email in the payment form card template so that they can be used when confirming the card payment intent

jagui avatar Jan 13 '20 12:01 jagui

Thank you for the PR.

Could you please tell us why you need the email ? Is it for tracking to display something to the customer ? Do you think others data could be interesting ?

clotairer avatar May 03 '20 08:05 clotairer

Hi @clotairer this PR allows to easily cross reference orders between the PS backoffice and the Stripe dashboard. This email is displayed in the Customer column in the dashboard and the name when you drill down into the transaction details. Very helpful also for quickly spotting fraudulent payments with very dissimilar emails and usernames

jagui avatar May 04 '20 17:05 jagui

Hi @jagui

The customer details in the Stripe Dashboard is to manage the save card (and retrieve feature) The next release (2.1.0, now in develop) will include this feature. That's why we have conflicts with you branch. A new table on the ps database will store the id customer and the customer stripe key to retrieve or update data. I think we need to wait the next release.

Thank you.

clotairer avatar May 05 '20 06:05 clotairer

Fair enough. Do you have an ETA for version 2.1.0? Cheers, Juan

jagui avatar May 05 '20 07:05 jagui

Tests are OK, we wait for translations (EN and FR is OK). And we are looking for beta testers if you want.

clotairer avatar May 05 '20 07:05 clotairer

I can definitely help w/ ES translation and I might look into beta testing, though I'm pretty busy developing a private module

jagui avatar May 05 '20 08:05 jagui

Hi @jagui

2.1.0 is now released and published on PrestaShop addons. You can activate save card in order to save customer details on Stripe Dashboard.

clotairer avatar Jun 04 '20 17:06 clotairer

T

Hi @jagui

2.1.0 is now released and published on PrestaShop addons. You can activate save card in order to save customer details on Stripe Dashboard.

HI again @clotaire202. I've already tested the 2.1.0 version, but I'm still experiencing the same problem. I'm sure it works fine with regular checkout flows, but with one-page-checkout the info about the customer on hookHeader is not set yet, and it needs to be updated once the customer has been created. I'm doing the update via the payment_form_card.tpl On top of that, I've noticed that the payment intent confirmation is failing due to an invalid ISO 2-letter country code (which happens when the customer's address is not yet set and the value passed by hookHeader is false.

Hope you can reconsider this. Cheers, Juan

jagui avatar Jun 05 '20 16:06 jagui

I'll check you proposal. Do you have a module OPC I can use to understand your issue ?

Could you have look on our refactoring too ? https://github.com/202-ecommerce/stripe_official/blob/feature/2.2.0/_dev/js/payments.js In this new version, we will call payment intent after submitting payment. It could solve your problem or get worse...

clotairer avatar Jun 05 '20 16:06 clotairer

Have you tried with the buit-in 1 page checkout flow? I use this module, which btw will have to adapt to https://github.com/202-ecommerce/stripe_official/pull/3

Also please note that you're basically doing the same trick with the stripe_amount variable: it might have changed since the header was sent, therefore you update it.

Anyhow I'll take a look at the changes you sent. Thanks for the prompt response, I appreciate your time.

Cheers, Juan

jagui avatar Jun 05 '20 16:06 jagui

Hello jagui, Sorry for our late reply, we have updated the module several times since we last talked. Can you update the module with the latest version (2.3.1) and tell us if your problem persists.

mribeiro202 avatar May 05 '21 09:05 mribeiro202

This PR is now out of date. We might close this PR.

Many thanks for your contribution.

clotairer avatar Dec 19 '22 08:12 clotairer