commerce icon indicating copy to clipboard operation
commerce copied to clipboard

vendure custom checkout support #619

Open hangai247 opened this issue 2 years ago • 11 comments

hangai247 avatar Mar 17 '22 09:03 hangai247

@hangai247 is attempting to deploy a commit to the Vercel Solutions Team on Vercel.

A member of the Team first needs to authorize it.

vercel[bot] avatar Mar 17 '22 09:03 vercel[bot]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployments, click below or on the icon next to each commit.

commerce-local – ./site

🔍 Inspect: https://vercel.com/vercel-solutions-vtest314/commerce-local/7doV2szZceyEuorQRaHSasVSkhrW
✅ Preview: https://commerce-local-git-fork-hangai-d94eab-vercel-solutions-vtest314.vercel.app

vercel[bot] avatar Mar 17 '22 09:03 vercel[bot]

Relates to #619 @okbel FYI this was done in coordination with me in the Vendure slack. Soninkhangai has agreed to take on some maintenance of this integration 👍

michaelbromley avatar Mar 17 '22 11:03 michaelbromley

Hey @michaelbromley

Is there an ETA for this feature to be merged?

cc: @hangai247

briandiaz avatar May 26 '22 05:05 briandiaz

@briandiaz I don't know, I'm not a maintainer of this repo so I cannot decide that unfortunately 😢

michaelbromley avatar May 26 '22 07:05 michaelbromley

@hangai247 Could you please add a Description to the PR to understand all you're adding please? We can review it and merge it.

okbel avatar May 26 '22 12:05 okbel

@michaelbromley Does this have your approval from the Vendure perspective? All approved practices?

okbel avatar May 26 '22 12:05 okbel

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
commerce-commercejs ✅ Ready (Inspect) Visit Preview Aug 17, 2022 at 2:06PM (UTC)
commerce-ordercloud ❌ Failed (Inspect) Aug 17, 2022 at 2:06PM (UTC)
commerce-shopify ✅ Ready (Inspect) Visit Preview Aug 17, 2022 at 2:06PM (UTC)
commerce-swell ✅ Ready (Inspect) Visit Preview Aug 17, 2022 at 2:06PM (UTC)
commerce-vendure ❌ Failed (Inspect) Aug 17, 2022 at 2:06PM (UTC)

vercel[bot] avatar May 26 '22 12:05 vercel[bot]

@okbel I'm just pulling this PR in locally to try it out. Unfortunately just had to spend ages trying to resolve https://github.com/vercel/commerce/issues/674 (in the end I used the suggested fix at https://github.com/vercel/commerce/issues/681#issuecomment-1041921734 to get it working). Will update this PR with my feedback.

michaelbromley avatar May 26 '22 14:05 michaelbromley

Oh that was quick. Thanks guys!

briandiaz avatar May 26 '22 16:05 briandiaz

@okbel ok I ended up having to make a bunch of fixes. @hangai247's work on this PR is excellent, so it was mainly just polishing up a few things and handling some cases in the checkout flow that weren't already covered. This is in a new PR at #760.

michaelbromley avatar May 26 '22 19:05 michaelbromley

Hey! Following up here. Is this something you're still interested in seeing through?

leerob avatar Nov 23 '22 03:11 leerob

Hi @leerob. I'd suggest this PR be closed in favour of #760.

michaelbromley avatar Nov 23 '22 08:11 michaelbromley