202-ecommerce / stripe_official

After years of hard work with Stripe connector for PrestaShop, 202 ecommerce stop the development of Stripe module on January 9th 2023. Thanks for all contributors that help us!
20 stars 20 forks source link

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

Closed jagui closed 1 year ago

jagui commented 4 years ago

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

clotairer commented 4 years ago

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 ?

jagui commented 4 years ago

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

clotairer commented 4 years ago

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.

jagui commented 4 years ago

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

clotairer commented 4 years ago

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

jagui commented 4 years ago

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

clotairer commented 4 years ago

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.

jagui commented 4 years ago

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

clotairer commented 4 years ago

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...

jagui commented 4 years ago

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

mribeiro202 commented 3 years ago

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.

clotairer commented 1 year ago

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

Many thanks for your contribution.