# 9월 7일 View 팀 회의 내용

## 각 Feautre별 진행 상황 공유와 병목이 발생하는 부분
예) 데이터 가공까지만 완료, 이벤트 핸들러 등록(인터렉션)만 남음, 다른 Feature와의 병합만 남음

1. Temporal Filter
(1) Line Chart
(2) Filter Box
현재 진행 상황은 (1) 데이터 가공 완료, (2) 라인 차트는 민정님께서 어제 해결함
Filter Box의 경우, 브러싱을 정혜님이 만듬. Line Chart가 이번주 내로 완료될 것 같다
라인차트를 이번주 완성되면 브러싱 넣을 것
브러싱 한후 필터링된 데이터를 다른 컴포넌트에 공유하는 것 역시도 해야함
Temporal Filter로 필터링된 데이터는 fake-asset이랑 같은 타입으로 만드는게 좋을 듯 합니다.

2. Vertical Cluster
(1) Cluster Graph
- v1: ClusterSize에 따라 height가 증가하도록 만들어, 오른쪽에 cluster에 포함된 모든 commit list를 보여줄 수 있도록 구현함.
- v2: ClusterSize를 내부 rect의 width로 표현하여, 오른쪽에는 전체 commit list가 아닌 cluster summary를 나타낼 수 있도록 구현함.

(2) Summary
- v1: 하나의 cluster안에 포함된 commit를 모두 보여주고
- v2: 모든 commit을 안보여주고 summary를 보여주고, cluster에 width도 적용됨 마지막 커밋을 summary로 보여주기보다는 commit에서 가장 많이 언급된 앞단어를 보여줌 (이 부분 PR [#97](https://github.com/githru/githru-vscode-ext/pull/97) 에서도 언급한 부분이니 꼭 봐주세요 !!)
- 고민하는 부분. (1. content 부분에서 커밋 메시지 키워드를 count하여 많이 나온 단어부터 보여주도록 하였는데 여기서 font size 등으로 style에 변화를 주는 것이 괜찮은 부분인지 2. author 부분에서 name마다 background-color를 다르게 주고 있는데 비슷한 색이 나오지 않도록 어떻게 처리할지)

3. Statistics
(1) Icicle tree
리뷰 반영 + 데이터 가공 + 상태 연동만 남았음.
(2) Bar chart
툴팁, 포커스했을 때 안에 데이터 보여지기, 변경됐을 때 transition 추가하기 이외에는 반응형 등추가

4. Detail Component
- scss (레이아웃 잡기)
- Vertical Cluster data 주입 ( 병합 )
- 데이터 가공 완료
유틸 함수가 많아지다보니 코드가 난잡해지는게 아닌가 싶어서, 리뷰를 꼭 받고싶습니다!!!!
서머리 바로 아래에 디테일 컴포넌트가 나타나는게 어떨까? -> 의견 나누고 싶은 부분에서 작성예

(영택님) **오늘 내일까지 모든 feature가 나와야 14일 배포가 가능할듯 합니다**. 현재 많이 빠듯함. 성현님이 같이 하시는게 좋을 것 같다. 대신하는 건 맞지 않고, 페어 프래그래밍으로 하는 것을 추천한다. UI, UX 고민하는 부분은 영택님이 뒤에 끼어서 다시 이야기하기로 할게요. 라인 차트가 중요한게 아니라 필터링이 중요함. 다른 분들은 필터링만 보고있지 말,고 fake-assets을 그대로 내려온다고 생각하고 작업을 해도 된다고 생각합니다. 전혀 의존성이 걸릴일이 없다고 생각합니다.

각 컴포넌트별로 위치 잡는 것에 대해서는 추가적으로 의견을 나누어야 할듯합니다.

## 의견 나누고싶으신 부분
1. 영후 - 컬러 통일과 관련된 내용 (김영후가 내용 정리해서 올릴 예정입니다.)
2. 영후 - convention과 관련한 논의(convention 영역이 아닐수도 있음)가 필요한 부분을 discussion에 리스트업하고 주단위로, 정말 가볍게 이야기 해보는 것은 어떨까요? [여기](https://github.com/githru/githru-vscode-ext/discussions/99)에 대충 리스트업 해놓았읍니다.

4. 영진 - Detail Component의 행방
-> Summary 밑에 Detail Component 넣기
6. 지혜 - PR 후 머지를 어느 때에 해야할지 궁금합니다 ... ! 그리고 스쿼시 머지 얘기도 좋을 것 같아요.
-> (영택) squash merge는 엔진팀에서 결정한것임. 따라도되고 안따라도됨. githru가 해결해줄 것이기 때문에 어떻게 해도 상관은 없음. PR은 approve 두명한테 받으면 끝내면 됩니다. 기다리지말고, 커멘트 쏟아진 것들은 무시하면 안되고 답은 달아야되고, 다음 PR에 올려도 되고 안올려도 되고, 2명 승인만 되면 됨. 1.0이 안나온 상태라서 merge를 막기에는 개발 속도가 너무 더뎌집니다.
(한슬) PR 중요도를 넣는 것은 어떨까요? -> (영택) 속도 측면에서 저하될 것 같아서, 안하는 편이 좋을 것 같습니다. 개인이 P1이라고 생각하는 것이 다른 사람은 P1이라고 생각 안할수 있기 때문에,

8. 기한 - 각 컴포넌트별로 위치 잡는 것에 대해서는 추가적으로 의견을 나누어야 할듯합니다.
(영택님) 라인차트 두개는 위에 박고, 디테일 컴포넌트는 커밋 밑에 넣고(아얘 맨 밑에 있는 것은 정보 링크가 잘 안된다고 생각합니다), 스크롤은 버티컬 클러스터 리스트만 스크롤, 통계는 그대로 가도 될듯합니다. 시간이 되면 반응형하는데, 가변이되는 부분은 cluster 와 summary 부분이고 나머지는 fixed. 반응형은 우선 나중으로,

## 14일까지 해야할 일 및 기타 해야할 일들
1. feature 끼리의 병합
- 금일 진행 예정 ( 3. 영진 - state handling 어떻게 진행할지?? ( 아래 1번과 동일 ))
-> PR 기다리는거 우선은 merge하고, 우선적으로 합치는 것 바로 진행하기.
-> 클러스터 그래프는 데이터만 바뀌면 클러스터 그래프도 자동으로 바뀜
-> summary도 데이터만 바뀌면 자동으로 바뀜.
-> 통계는 선택한 클러스터의 정보를 알아야 함.
-> 상태를 원본 상태와 필터링된 데이터 두가지를 가지고 있어야함.(정혜님이 작성할 예정)
-> 선택된 클러스터 정보 역시도 앱단에서 관리하여 statistics에 내려주어야함 (지혜님)
-> 통계에 무언가를 더 넣고싶을때 전체 기간에서 차지하는 비율 등, 다양한 데이터를 넣을 수 있음. 나중 이야기.

2. engine과의 병합
- engine팀 기능 구현이 완료되는 일정을 문의한 상태. 영택님이 문기님과 이야기하신 후 말씀해주신다고 함.
3. extension 배포 (14일)
4. 멘티 보고서 작성
- 개개인도 최대한 많이 올려주는게 영향이 간다고함. 정량적인 부분, 정성적인 부분(최대한 포장 잘하기!!!), PR Description에 올린것들도 잘 갈무리해서 작성해주세요! issue나 PR, discussion, wiki 여러가지 등은 영택님이 나중에 정리할 수 있는 문서를 따로 제공해주실 것.
6. 기타
- 클러스터 그래프쪽은 v1로 갈까 v2로 갈까? -> 투표 올릴 것. 컬러나 서머리에 키워드를 무엇을 보여줄 것인지, 디자인 스페이스 탐구가 필요한데 우선은 나중에.

## 14일 이후부터 할일
이후에 이야기하기

## 다음 만남 일정은 언제?


영진님이 11일날까지는 영택님께 병합된 것 전달하실 예정.
14나 15때 날 잡아서, 영택님께서 한번 공지할 예정. 오프라인이나 온라인은 고민중이십니다.
레이아웃 잡기는 기한님께서

implicit branch,seq, isMergeCommit은 아무도 안씀. taskId는 씀.taskId 대신 clusterId로 바뀜.

# 다들 추석 잘보내세영~~

# Fire~~~!
깃쓰루 왕 최고 :+1: