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

feature/cart: GET server-side & PUT api hook #66

Merged
merged 3 commits into from
May 16, 2024
Merged

Conversation

winniehuang0616
Copy link
Collaborator

🤔 This is a ...

  • [v] feat: add feature
  • fix: bug fix
  • docs: documentation
  • style: formatting, missing semi colons
  • refactor: rewrite codes
  • test: add tests
  • chore: maintain, .gitignore or workflow
  • env: environment setup or change
  • other: about what?

🔗 Related issue link

  • 我不確定 cart 那邊是要抓 session.user.id 還是會有 parameter,就先照本來的 cart/[cartId],但目前看起來應該不是這樣,所以可能要先直接改成有 cart id 的 url,拍謝,實在有點沒時間ㄌ嗚嗚嗚

💡 Background and solution

☑️ Self-Check before Merge

  • [v] yarn lint
  • yarn e2e
  • yarn test

@guan404ming guan404ming merged commit ad37cee into main May 16, 2024
4 checks passed
@guan404ming guan404ming deleted the cart-with-api branch May 16, 2024 07:33
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