Skip to content
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

Added Umbraco provider #1170

Merged
merged 3 commits into from
Aug 23, 2023
Merged

Conversation

mattbrailsford
Copy link
Contributor

@mattbrailsford mattbrailsford commented Aug 11, 2023

Adding a link to the Umbraco + Umbraco Commerce implementation of the Vercel Commerce demo.

Worth noting, our repo has a little different structure as we wanted to include the back end project as well so we have a backend/main and a frontend/main branch accordingly. The frontend/main branch can easily be merged with upstream changes from the vercel commerce repo.

NB This is a re-submission of #1165 as during some cleanup I inadvertently removed my previous fork 🤦‍♂️

@vercel
Copy link

vercel bot commented Aug 11, 2023

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

A member of the Team first needs to authorize it.

@mattbrailsford
Copy link
Contributor Author

@leerob just to follow up from the original PR, the 500 error is resolved and I'd like to request a copy of the product images to see if that resolves slow image load speeds 👍

README.md Outdated Show resolved Hide resolved
@manovotny
Copy link
Collaborator

I'd like to request a copy of the product images

You can download them here.

@mattbrailsford
Copy link
Contributor Author

Images should now be updated so hopefully this resolves the slow loading for you.

Beyond that, I believe it should now be good to approve

@vercel
Copy link

vercel bot commented Aug 23, 2023

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

Name Status Preview Comments Updated (UTC)
commerce-shopify ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 23, 2023 3:28pm

@manovotny manovotny merged commit 18167d2 into vercel:main Aug 23, 2023
2 checks passed
@manovotny
Copy link
Collaborator

@mattbrailsford thanks for all your work on this! 🙏

@mattbrailsford
Copy link
Contributor Author

No problem at all. Thank you for taking the time to feedback. It’s a lovely demo project and really glad I got to implement it.

bc-alexsaiannyi pushed a commit to bigcommerce/nextjs-commerce that referenced this pull request Dec 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants