-
Notifications
You must be signed in to change notification settings - Fork 0
2차 스프린트 리뷰 & 회고
Sang In Chun edited this page Aug 23, 2021
·
1 revision
- 메인 페이지 UI (+캐러셀)
- 헤더 UI 및 최소 기능
- 상품 상세 페이지
- 이미지 확대보기
- 썸네일 이미지 바꾸기
- 리뷰 UI
- 상품 목록
- 필터링, 정렬 정상 동작
-
sale
, (best
,new
) 태그 - 세일 시 UI 표시
- Option Store
- product list를 불러오는 option들을 관리하는 store
- 상품 목록 API
- 에러 미들웨어
- 인증 미들웨어
- 데이터 SQL
- 찜하기/취소 API
- 상품 디테일 API
- 백엔드에서 데이터 검증
- 백엔드 레거시 에러 핸들링 제거
- 커서 대각선 이동
- 찜하기 UI 연동
- 리뷰 API + 연동 (POST)
- 장바구니 (테스트코드, PR리뷰, 머지 필요)
- 옵션/수량 변경
- 검색 UI
- 최근 검색 드롭다운
- 검색 API(간단히 Like) > 추후 상황에 따라 변경
- 로그인 처리 필요
- 현재까지 각자 개발한 기능들 머지되면 합치는 작업
- 라우트 연결, 컴포넌트 붙이기 (더미로라도)
- UI 수정
- API 호출 / 에러 처리
- 기능 데드라인을 만들어놓고, 그 이후로는 추가 기능 개발보다는 에러처리에 집중해봐도 좋을 듯
- 테스트코드 강제로라도 작성하니까 좋았음
- 간단한 컴포넌트라도 테스트 있으니까 실수 줄이는 데 도움됨
- 바쁜 일정 중에도 PR 리뷰 아직까지 열심히 하고 있는 것 같음
- 모르는 부분 서로 물어보고 토론(?)하고 알려주고 하는 분위기 좋았음
- React 상태관리에 대해 잘 몰랐는데, 이번에 mobX 쓰면서 왜 필요한 건지 어떻게 하는지 이런 부분 알게 돼서 좋았음
- 저번 주보다는 좀 더 친해진 거 같아서 좋았음 (불화가 없어서)
- PR 올리면 최대한 빨리 확인하고 리뷰하기 (시간을 딱 정하기보다는 미루지는 않는 걸로..)
- 이슈를 나눈다고 나눴지만 그래도 여전히 단위가 컸던 것 같다..
- 개발하다보니 추가되는 (미리 예상하지 못했던) 필요한 기능들......어렵다
- 써브 Task를 이슈로 만드는 기능도 조금 활용해보면 좋다
- 큰 이슈에 assignee 한 명 -> 작은 이슈 여러 개로 나누기 -> PR은 작은 단위로
- 이런 식으로 하는게 좋지 않을까?
- 큰 이슈 각각은 독립적으로
- 작은 이슈들은 연속된?..