4주차 피어세션 26그룹

Jump to bottom

dbstjrwnekd edited this page Dec 11, 2020 · 1 revision

4주차 피어세션 - 26그룹

🔥 참여 캠퍼

A

J128_윤석주

C

J210_최홍일

D

J004_강민수

E

J029_김도연

F

J074_박동현, J209_최창희

🕖 활동 과정

프론트 권한 인증은 어떻게 해야할까?

D조는 현재 프론트 페이지를 렌더링할 때 마다 인증체크 api를 날리고있는데, api 호출을 줄이기 위한방법이 있을까요..?
- app.tsx 안에서 로그인 상태가 변경될 때만 useEffect 사용하기
- 처음 로그인 시에 권한정보를 상태로 관리하기

다음주에 기능 개발? 리팩토링?

A
- 리팩토링을하며 서로의 코드를 이해하는 시간을 가질것 같습니다. 자잘한 버그도 수정을 해야 할 것 같습니다.
C
- 리팩토링, 서로의 생각을 다 알 수 있게 대화하기
D
- 화요일 또는 늦으면 수요일까지 기능 구현 그 이후에는 버그잡고 리팩토링하기로 하였습니다.
E
- 필수 기능 구현을 제외하고 남은 시간은 리팩토링을 할 것 같습니다.
- 부가 기능은 6-7주차 개발을 고려중입니다.
F
- 기존에 구현한 기능의 동작을 새로운 방식으로 변경해야해서 리팩토링 겸 새로운 개발이 진행될듯 싶어요...

코드 리뷰는 어떻게?

A
- 매 PR마다 코드리뷰를 하고 있습니다. 컨벤션은 괜찮은지, 구조는 간단하게 했는지, 하드코딩을 하지 않았는지, 논리는 괜찮은지 등을 팀원 모두가 확인합니다.
- 최소 2명 이상이 커밋해야 merge가 가능합니다.
- PR시 주요 논리가 들어있는 코드를 간단하게 설명하고, 실행화면을 첨부하여 리뷰를 돕고 있습니다.
C
- 시간 날때마다 하는편
- 요즘은 작업이 많아서 잘 안지켜진다.
- 아침에 스크럼시간에 보통 자기가 작업한 내용을 보여주면서 설명함
D
- 오전에 1시간~2시간정도 시간을 잡고 PR 리뷰를 하고 있습니다.
- 리뷰를 하면서 코딩 컨벤션을 많이 업데이트 시켰습니다.
E
- 코딩컨벤션을 준수하는지
- 하드코딩하지 않았는지
- 변수명을 올바르게 선언하였는지
- 새롭게 적용한 문법의 사용법은 어떻게 되는지
- 등의 내용에 대해 자유롭게 댓글을 남깁니다.
- 매 PR마다 가능한 모든 팀원이 댓글을 남겨야 merge하는 방식으로 진행하고 있습니다.
- 하루에 2-3회 정도 코드리뷰 시간을 갖습니다.
F
- 최소 2명 이상이 커밋해야 merge
- 개선점이 있는가
- 리뷰 도중 의논거리가 있으면 다같이 회의
- 오탈자나 컨벤션 위주로 먼저 확인하고 테스트용 코드가 남아있지 않은지 체크합니다
- 코드의 흐름을 쭉 따라가보면서 이해가 잘 안되는 부분을 질문

Store? State?

A
- 컴포넌트간의 공유할 데이터가 있다면 store,없다면 컴포넌트딴에서 가지고 있는게 좋을 것 같습니다.
C
- 전역에서 공유할 일이 있다면? Store 아니라면? State
- 둘 중 어느 하나를 사용하더라도, 왜 사용했는지가 중요하다.
F
- 전역관리가 필요할때만

발표 주제는 ...?

A
- 논의예정입니다.
C
- 다음주에 얘기해보지 않을까요?
D
- 기술적인 도전이 많이 없어서 협업과정에 대해서 발표해보려고 논의중입니다.
E
- 주말에 논의할 것 같습니다.
F
- 금일 정규시간 이후 논의

부캠하면서 배운것

전반적인 웹개발과 팀으로 일하는 방법을 배운 것 같습니다.
부캠 이전에 혼자 공부할때는 할 수 없었던 팀원들과의 작업을 통해 협업하는 방법을 익힐 수 있었습니다.
백로그, api명세, 기획서 등 팀끼리 작업하기 위해 필요한 지식들을 익힐 수 있었습니다.
깃헙의 다양한 기능
전반적인 웹 생태계, 지식
커뮤니케이션 주도의 중요성
기획-개발-배포의 과정을 모두 경험할 수 있어서 좋았습니다.
개발지식을 떠나서 팀으로 협업하는 방식을 가장 많이 배운것 같습니다.
동료들의 학습방법, 개발에 대한 태도에서 앞으로 나아갈 방향성을 찾음
개발 얘기를 나눌수 있는 친구(?)들이 생겨서 너무 행복 😀