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

8주차 실습 완료 #136

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

8주차 실습 완료 #136

wants to merge 6 commits into from

Conversation

holdn2
Copy link
Contributor

@holdn2 holdn2 commented Nov 22, 2024

📚 워크북 링크

https://alive-wing-8df.notion.site/8-1422a5d201b980adad29d840cdf43c4e?pvs=4

📝 변경/추가 사항 요약

비밀번호 입력 플로우 실습 완료.
data fetching 적용해서 이전 미션 수정. (OrderBar, Stores, Store, Cart 화면 서버에서 가져온 정보로 적용)

✅ 리뷰 요구사항

  1. 7주차에 진행했을 때 만들었던 함수들이 강의와 다른 부분이 있어서 복잡하게 구현한 것 같은데 리뷰 부탁드립니다.
  2. addMenu를 할 때 POST를 사용하면 동일한 store의 메뉴를 담을 때도 id가 다른 것으로 한 개 더 생성되는 문제가 있어서 addMenu시 db의 cart에 어떤 것이 있으면 모두 DELETE 후 새로운 정보로 POST하게 만들었습니다. 좀 많이 복잡해진 것 같아서 리뷰 부탁드립니다.

📸 확인 방법 (스크린샷)

image
image
image
image
image
image
image
image
image
image
image
image
image

@holdn2 holdn2 self-assigned this Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant