-
Notifications
You must be signed in to change notification settings - Fork 5
회의록 5주차
Seongbin Kim (김성빈) edited this page Dec 3, 2018
·
2 revisions
작성: 프론트엔드팀 일꾼 - 김성빈(@Ethan
)
작성일: 2018-12-03 AM 9:20
-
1주 동안 했던 작업 공유
튜토리얼 진행, 테스트 공부
-
wireframe 분석을 통한 컴포넌트 단위 분석 및 작업 분배
-
협업 이슈
- 디자인팀: 주문 확인 페이지의 Conter의 디자인
- 백엔드팀: Cart 관리 주체 여부
-
🚀 TODOLIST 🚀
개인별 컴포넌트 작성
-
1주 동안 했던 작업 공유
-
[General] 토의 내용
-
[그룹제 폐지] 개발 기간이므로 그룹 폐지
-
[다음주 세션 예고] Javascript, React Test/TDD Live Demo를 진행하기로
@Ethan
-
[Redux-Thunk vs Saga]
- Thunk를 배우는 것이 현재 deadline을 지키면서 가능할 것 같지 않다. Thunk는 튜토리얼에서 자주 접하고 간단하므로 Thunk로 Redux Async를 다루면 될 것 같다.
-
[브랜치 전략에 관하여]
@Ethan
- master <--merge-- develop <--commit/push-- 보다,
- master <--merge-- develop <--merge-- feature <--commit/push-- 가 낫다고 생각
-
이슈 생성:
페이스북 소셜 로그인 버튼 커스터마이징 컴포넌트 구현
- 이슈번호도 생성됨 (#3) > 구현 중복 / 브랜치 중복등을 피하기 위해Assignee
에 본인을 되도록 등록해주세요. 아니면 브랜치 생성 전 매번 확인해야 할 수도 있습니다. -
브랜치 생성 (local):
f#3-fb-social-login-btn-custom-component
- 브랜치명은f#이슈번호-구현주제로 네이밍
- 커밋 진행 (commit): 본인의 로컬 feature 브랜치에 commit
- 리모트 저장소에 푸시 (push): 동일한 브랜치를 리모트에 생성 및 동기화 완료
-
Pull Req 생성 (github):
f#3-fb-...
브랜치를develop
브랜치로 merge하기 위함 - Pull Req 코드 리뷰 (github): 다른 멤버들이 리뷰
- Pull Req merge (github): Pull Req merge 완료
- 이슈 닫기 (github): 이슈 닫음
-
[협업 이슈] 주문 확인 페이지의
Counter
컴포넌트 관련해서@디자인팀
과 논의 - UI는 배민처럼 하기로!배민 앱은 주문 확인 시 옵션 밑에 카운터가 있고, 현재 wireframe은 옵션 위에 있어 혼동의 여지가 있음
-
[협업 이슈] API 프로토타입 관련해서
@백엔드팀
에 요청 - Cart는 프론트에서 관리하기로 -
[개발] 컴포넌트 단위 분석 및 개발 방법에 대해 간략히 논의
- (ex) Header 컴포넌트 - backBtn과 title만 작동하면 됨 - 햄버거는 버튼만
- (ex) Counter 컴포넌트 - number를 내부 상태로 갖는 형태로 개발
-
-
다음 주 Todolist 결정
- 개인별 컴포넌트 할당
-
@Rachel.Lee (공통 컴포넌트)
- Header (back, hamburger, title)
- Footer
-
@John.Lee (공통 컴포넌트)
- Counter
-
@Kyun (상품 목록[Catalog] 부분)
- CategoryNavBar
- CategoryHeader
- CatalogItem (= MenuItem)
-
@Leo (가게[Store] 프로필 및 상품 상세 정보 부분)
- StoreProfile
- MenuProfile
- MenuOptionItem
-
@Hosung (장바구니[Cart] 부분)
- CartMenuItem
- CartOptionItem
-
@Rachel.Lee (공통 컴포넌트)
- 개인별 컴포넌트 할당