-
Notifications
You must be signed in to change notification settings - Fork 217
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
feat: (fix too) Migrate to Stripe IntentPayment API #20
base: develop
Are you sure you want to change the base?
Conversation
fix tab error fix tab error fix tab error fix tab error fix tab error try to migrate to payment intents api migrate to paymentintents api migrate to paymentintent api migrate to paymentintent api migrate to IntentPayments API migrate to IntentPayments API migrate to IntentPayments API migrate to IntentPayments API migrate to IntentPayments API migrate to IntentPayments API migrate to IntentPayments API migrate to IntentPayments API migrate to IntentPayments API migrate to IntentPayments API migrate to IntentPayments API Migrate to PaymentIntent API Fix CSRF Token Error due to frappe not ready in creation of PaymentIntent when a user is connected Migrate to Payment Intents API, Payment Intents OK, validation OK, TODO : link PI and PR, create payment in ERPNEXT Migrate to PaymentIntent API : create Payment Entry whe payment suceed enhance paymentintent enhance paymentintent enhance paymentintent enhance paymentintent enhance paymentintent enhance paymentintent enhance paymentintent enhance paymentintent enhance paymentintent enhance paymentintent enhance paymentintent enhance paymentintent enhance paymentintent enhance paymentintent enhance paymentintent enhance paymentintent enhance paymentintent enhance paymentintent enhance paymentintent enhance paymentintent enhance paymentintent enhance paymentintent enhance paymentintent enhance paymentintent enhance paymentintent enhance paymentintent enhance paymentintent enhance paymentintent fix indent error fix indent error enhance paymentintent validation enhance paymentintent validation enhance paymentintent validation enhance paymentintent validation enhance payment validation enhance payment validation enhance payment validation enhance payment validation enhance payment validation enhance payment validation enhance confirmation of payment intents enhance payment intent confirmation enhance payment intent confirmation enhance payment intent confirmation enhance payment intent confirmation enhance payment intent confirmation enhance payment intent confirmation enhance payment intent confirmation test in enhance payment intent confirmation test in enhance payment intent confirmation test in enhance payment intent confirmation test in enhance payment intent confirmation test in enhance payment intent confirmation test in enhance payment intent confirmation test in enhance payment intent confirmation test in enhance payment intent confirmation test in enhance payment intent confirmation test in enhance payment intent confirmation finalize enhancement of payment intent confirmation and revue of stylesheets etc. reuse of showMessage function to display errors in submitting form redirect after success delete payment-success.py delete payment-success.py
@MorezMartin Do you think this might fix #39? I tried to buy an item costing above 100 β¬. Did you receive the same error message? |
@Spixmaster not sure it is, i think your problem is more related to the configuration of your Payement Gateway. |
@MorezMartin Thank you. I think I found the solution. The document could not be found for whatever reason. It probably had nothing to do with Stripe. I recreated the payment gateway and it worked. Here is a similar issue, frappe/erpnext#16928. Nevertheless, this pull request should really be merged. It is the proper way of working with Stripe. |
Any progress on this pull request? This will have a very positive impact on the software, in my country it is not common that people pay with a credit card |
Merge conflicts after minor updates, i have not try this version yet since webshop is a separate repo, i will try it and modify if necessary |
tested, version working, please merge |
Any update on getting this merged? I'm in urgent need of this. |
Migrate to IntentPayment API:
Keeping the automatic validation of Payment Entries and Sales Invoices.
Have to add a payment_id field in Payment Request in ERPNext (maybe it can be achieve with this repos but i didn't success on this point)
Thanks in advance,
Sincerely yours,
Martin Morez
Squashing commits: