feat(medusa): Update payment session management

[adrien2p]

What

Update the session management in the cart checkout flow.

[changeset-bot]

🦋 Changeset detected

Latest commit: 7397483

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@medusajs/medusa	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[adrien2p]

I ll perform some tests through my scripts. We should also plan some tests on staging around this chore

[adrien2p]

From the tests, we can see that only one payment is created for the active session instead of one payment for each session no matter the one that the customer selects. We can also see that only once customer is created as well as one intent and this intent is updated twice, one for each cart manipulation post session selection

[adrien2p]

and one of the payment is cancelled when it is not selected anymore

[adrien2p]

/snapshot-this

[adrien2p]

I ll continue the test locally for now

[github-actions]

🚀 A snapshot release has been made for this PR

Test the snapshots by updating your package.json with the newly published versions:

yarn add @medusajs/medusa@1.7.3-snapshot-20230106120456

yarn add @medusajs/medusa-js@1.3.6-snapshot-20230106120456

yarn add @medusajs/stock-location@2.0.0-snapshot-20230106120456

Latest commit: 39c834e

[adrien2p]

/snapshot-this

[github-actions]

🚀 A snapshot release has been made for this PR

Test the snapshots by updating your package.json with the newly published versions:

yarn add @medusajs/medusa@1.7.3-snapshot-20230106122348

yarn add @medusajs/medusa-js@1.3.6-snapshot-20230106122348

yarn add @medusajs/stock-location@2.0.0-snapshot-20230106122348

Latest commit: 39c834e

[chemicalkosek]

I've done some payment tests on this snapshot with multiple stripe providers. All went fine :)

[adrien2p]

@olivermrbl for the existing payment session, I wanted to add a migration that set them as is_initiated if there have some data? What do you think? should I proceed?

[olivermrbl]

@adrien2p Yes, I think that is an excellent idea 👍

[olivermrbl]

Do you want to add it in this PR or a subsequent one?

[adrien2p]

Do you want to add it in this PR or a subsequent one?

I will add it in that one 👌e

[olivermrbl]

@srindom Do you have any final comments before we merge?

[olivermrbl]

LGTM, strong work! 🚀