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

Replaces Route Handlers with Server Actions #1050

Merged
merged 2 commits into from
Jun 17, 2023
Merged

Replaces Route Handlers with Server Actions #1050

merged 2 commits into from
Jun 17, 2023

Conversation

manovotny
Copy link
Collaborator

@manovotny manovotny commented Jun 16, 2023

This replaces the POST, PUT, and DELETE handlers with Server Actions`.

This affects the following actions in the app:

  • Adding items to the cart
  • Removing items from the cart
  • Updating item quantity in the cart

Closes #1018
Fixes #1011, #989

@vercel
Copy link

vercel bot commented Jun 16, 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 Jun 16, 2023 10:04pm

@manovotny
Copy link
Collaborator Author

manovotny commented Jun 17, 2023

This is ready for review. I noticed an existing issue with the cart that I was going to try and also fix in this PR, but decided to not to, otherwise it would get too noisy with unrelated changes.

The issue, which is not due to these changes, it logged in #1051.

@leerob
Copy link
Member

leerob commented Jun 17, 2023

I noticed the same bug, good call out 👍

@leerob leerob merged commit 585b3bb into main Jun 17, 2023
@leerob leerob deleted the server-actions branch June 17, 2023 18:18
bc-alexsaiannyi pushed a commit to bigcommerce/nextjs-commerce that referenced this pull request Jul 21, 2023
bc-alexsaiannyi pushed a commit to bigcommerce/nextjs-commerce that referenced this pull request Dec 7, 2023
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.

Convert to use Server Actions
2 participants