medusa icon indicating copy to clipboard operation
medusa copied to clipboard

fix #6438

Open AxeemHaider opened this issue 1 year ago • 2 comments

AxeemHaider avatar Feb 19 '24 16:02 AxeemHaider

⚠️ No Changeset found

Latest commit: dd3538dc6afac68a7420ea1b48abc88039a90b1f

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

changeset-bot[bot] avatar Feb 19 '24 16:02 changeset-bot[bot]

@AxeemHaider is attempting to deploy a commit to the medusajs Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] avatar Feb 19 '24 16:02 vercel[bot]

Is there a plan to have this merged? I myself have had issues with auto capture of payment on the medusa side and this this would solve my issue. If there has been another solution or insight into why orders are not auto capturing please point me in the correct direction

SapioBeasley avatar May 28 '24 15:05 SapioBeasley

Could you please rebase your PR to v1.x? We also have to execute the method capture of the payment service to create the entity and trigger the event. cc: @olivermrbl @adrien2p

carlos-r-l-rodrigues avatar May 28 '24 17:05 carlos-r-l-rodrigues

Hey, thanks for the PR! Since v2 brought a lot of architectural changes, we will be closing this PR since it no longer applies to our new setup.

If you would still like to open this PR, please rebase with the v1 branch and re-request a review & we'll get back on it quickly.

riqwan avatar Jul 05 '24 12:07 riqwan