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

Bunch of messy updates #1

Conversation

dayhaysoos
Copy link

Hey @bhattrajat, so I'm not exactly sure how that error was coming up, but I updated your branch package.json using pnpm --filter nextjs-app-router use-shopping-cart@next to install the alpha version. The alpha version has the date-fns package installed.

I was able to run the project and I didn't see any errors related to date-fns. I haven't done too much digging to figure out why you got that error, though, because it simply should have not tried to refer to date-fns.

I also took the liberty to do a bunch of messy things in your fork.

Try running the project:

pnpm --filter nextjs-app-router dev

and tell me if it can run. Also are the images broken when you're looking at the cart items and what not?

Sorry this has been such a mess! I been busy and did so much in a hurry. Hopefully I didn't mess anything up

@bhattrajat bhattrajat changed the base branch from master to example-next-app-router October 15, 2023 20:19
@bhattrajat bhattrajat merged commit 1cb1825 into bhattrajat:example-next-app-router Oct 15, 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
2 participants