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주차] 실습 완료 #141

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

[8주차] 실습 완료 #141

wants to merge 12 commits into from

Conversation

jstar000
Copy link
Member

📚 워크북 링크

https://sand-knot-d9d.notion.site/8-142b555a2dff8098b780eb76bd897cee?pvs=4

📝 변경/추가 사항 요약

8주차 실습 완료

✅ 리뷰 요구사항

📸 확인 방법 (스크린샷)

- stores.js에 가게 정보를 요청하는 api 선언
- cart.js에 장바구니 정보를 요청하는 api 선언
- getStores 동작 확인 완료 / updateCart 동작 확인 완료
- 서버에 저장된 장바구니 데이터를 불러와 새로고침 후에도 유지되도록 구현
- 기존 강의 코드로 작성된 `fetchCart`가 정상적으로 동작하지 않아 일부 로직 수정
- 수정된 코드는 서버 데이터를 반영하며 장바구니 상태를 업데이트함
- PasswordPage 컴포넌트 생성: 현재 상태(비밀번호 입력 중, 입력 완료, 잘못된 비밀번호 입력 등)에 따라 뷰를 재사용할 수 있도록 설계
- PasswordPage가 PasswordInput 컴포넌트로 props를 전달하도록 구조 수정
- constants 폴더 추가: 비밀번호 길이를 저장하는 password.js 파일 생성
- utils 폴더 추가: 배열을 섞는 shuffle 메서드를 구현한 shuffle.js 파일 생성
- keypadNumbers의 shuffle() 메서드를 useMemo로 감싸 리렌더링 시에도 동일한 숫자 배열을 유지하도록 수정
@jstar000 jstar000 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