@jonathanshaw i have removed commit with change. Agree with you, it requires separate issue, with detailed descriptions of problem.
@jonathanshaw I added PaymentIntent::STATUS_REQUIRES_ACTION because if the user selects PayPal (or other method which create state "require_action"), the PaymentIntent enters this state. However, if they return without completing the payment and modify their order, we need to update the PaymentIntent amount accordingly.
I understand that this change might affect other payment methods. In this case we might need to provide more flexible logic. If you see any potential issues, let's discuss how to handle them properly.
gaydabura → made their first commit to this issue’s fork.
+1
+1
gaydabura → changed the visibility of the branch 3392479-roadmap-from-1.x to active.
gaydabura → changed the visibility of the branch 3392479-roadmap-from-1.x to hidden.
gaydabura → changed the visibility of the branch 3392479-roadmap-from-1.x to hidden.
hello @edysmp, after some research i can say its possible, we will implement this feature in next release. Thank you.
gaydabura → made their first commit to this issue’s fork.
gaydabura → made their first commit to this issue’s fork.
@andypost pls review https://git.drupalcode.org/project/component_connector/-/commit/fb737bc9...
@Hamid.ali hello, it does not fix the issue, also when you call parent constructor, you didnt pass all needed vars.
+1
fixed in 1.0.1
fixed in 1.0.1
+++
G4MBINI → credited gaydabura → .
Hello, @edysmp, this is expected behavior, here https://api.drupal.org/api/drupal/core%21lib%21Drupal%21Core%21Render%21...
Each information array must contain either a 'variables' element (for using a #theme element) or a 'render element' element (for render elements), but not both.
Thank's for good notice, we will add support in next minor release with other fixes.
G4MBINI → credited gaydabura → .
G4MBINI → credited gaydabura → .