View | 220907 Meeting Note
- 성현님이 민정님과 페어프로그래밍 할 예정입니다.
- Temporal Filter로 필터링된 데이터는 fake-assets이랑 같은 타입으로 만드는게 좋을 듯 합니다.
- Line Chart는 이번 주 내로 완료될 것 같습니다.
- cluster graph는 다음과 같이 구현한 상태입니다.
- v1: ClusterSize에 따라 height가 증가하도록 만들어, 오른쪽에 cluster에 포함된 모든 commit list를 보여줄 수 있도록 구현함.
- v2: ClusterSize를 내부 rect의 width로 표현하여, 오른쪽에는 전체 commit list가 아닌 cluster summary를 나타낼 수 있도록 구현함.
- summary는 다음과 같이 구현한 상태입니다.
- v1: 하나의 cluster안에 포함된 commit를 모두 보여주고
- v2: 모든 commit을 안보여주고 summary를 보여주고, cluster에 width도 적용됨 마지막 커밋을 summary로 보여주기보다는 commit에서 가장 많이 언급된 앞단어를 보여줌 (이 부분 PR #97 에서도 언급한 부분이니 꼭 봐주세요 !!)
- 고민하는 부분. (1. content 부분에서 커밋 메시지 키워드를 count하여 많이 나온 단어부터 보여주도록 하였는데 여기서 font size 등으로 style에 변화를 주는 것이 괜찮은 부분인지 2. author 부분에서 name마다 background-color를 다르게 주고 있는데 비슷한 색이 나오지 않도록 어떻게 처리할지)
- iclcie tree는 리뷰 반영 + 데이터 가공 + 상태 연동만 남았습니다.
- bar, chart는 툴팁 + 포커스 했을 때 안에 데이터 보여지기, 변경됐을 때 transition 추가하기 및 반응형 등의 추가가 남았습니다.
(영택님) 오늘(7일) 혹은 내일(8일)까지 모든 feature가 나와야 14일 배포가 가능할듯 합니다. 현재 많이 빠듯하니, 성현님이 같이 하시는게 좋을 것 같습니다. 대신하는 건 맞지 않고, 페어 프래그래밍으로 하는 것을 추천한다. UI, UX 고민하는 부분은 영택님과 다시 이야기하도록 합니다. 라인 차트가 중요한게 아니라 필터링이 중요한데, 그렇다고 다른 분들은 필터링만 기다리고 있지 말고, fake-assets을 그대로 내려온다고 생각하고 작업을 해도 된다고 생각합니다. 전혀 의존성이 걸릴 일이 없다고 생각합니다.
- 임시적으로 결정된 컬러 코드는 다음과 같습니다.
- body background-color : grey900
- main theme color : #0077aa
- sub color : blue-light-A700, blue-light-A200 이외에 사용하는 경우 파란색 계열로 통일되도록 하기
- vscode의 기본 컬러가 블루인데, githru 역시도 primary 컬러를 블루로 가져가면 통일성은 유지될 수 있지만 vscode의 hello world처럼 보일 수 있다. 일단은 블루 계열 컬러로 진행하되, 추후 바꿀 수 있게 변수화를 잘 시켜놓도록 합니다.
- default 컬러는 dark로 가져갑니다.
- light 모드의 지원은 여유가 되면 하는 것으로 하고, 우선은 dark를 기반으로 구현하는 것에 집중합니다.
- gitlens에서 vscode 테마 컬러가 반영되는 기능을 githru에도 역시 적용하는 것을 김영후님이 검토할 예정입니다.
convention과 관련한 논의(convention 영역이 아닐수도 있음)가 필요한 부분을 discussion에 리스트업하고 주단위로, 정말 가볍게 이야기 해보는 것은 어떨까요? 여기에 대충 리스트업 해놓았읍니다.
원래 Summary와는 별개로 아래쪽에 위치시키려 했으나, summary 밑에 넣는 편이 추후 반응형에 대한 대비 및 summary 정보와의 연관성(링킹)을 위해서 summary 바로 밑에 보여주는게 좋을 듯 합니다.
(영택님) approve가 2개 이상 적용되기만 하면 merge해도 상관 없습니다. 다만 커멘트가 쏟아지는 것들은 반드시 답글을 달아야하고, 해당 내용에 대해서 다음에 PR을 올릴때 반영해도되고, 반영하지 않아도 좋습니다. (한슬님) PR에 중요도를 넣는 것은 어떨까요? => (영택님) 속도 측면에서 저하될 것 같고, 판단 기준이 개개인마다 다르기 때문에 안하는 것이 좋을 듯 합니다.
(영택님) 라인차트는 맨 위에, 디테일 컴포넌트는 Summary 밑에, 버티컬 클러스터 및 서머리는 지금 위치를 차지하고, 통계 역시도 현재 위치로 가면 될듯 합니다. 시간이 되면 반응형을 하는데, 가변이 되는 부분은 cluster와 summary 부분이고 나머지는 fixed 될것으로 보입니다.
- (영택님) 현재(7일) PR올라온 것들은 우선적으로 merge하고, 합치는 것을 바로 진행하도록 합니다.
- state handling을 어떻게 할 것인지에 대해서 영진님께서 라이브 코딩으로 보여주었습니다.
(민기님) engine 기능 구현 완료(view만 붙이면 되는 수준)는 일요일 전을 목표로 하고 있습니다.
(영택님) 개개인이 작성하는 보고서의 정량적인 부분, 정성적인 부분 모두 평가 요소에 들어가며, 지금까지 활동한 내용들 잘 갈무리해서 작성해주세요. issue나 discussion, pr, wik 등 여러 곳에 활동 내역과 기록이 산재되어 있는데, 추후 한 곳으로 모으기 위한 문서를 따로 제공하려고 합니다.
(영택님) 컬러라던지, summary로 무엇을 보여줄지 등에 대해서는 추후에 이야기하도록 하고 우선은 구현에 집중합니다.
- 11일까지 컴포넌트 병합이 끝나야 영진님께서 영택님께 코드 전달 가능
- 영택님께서 14일 혹은 15일중에 모임 추진할 예정. 온라인, 오프라인은 고민 중. 구체적인 내용은 추후 공지올리실 예정
- 컴포넌트 병합과 관련해 전체적인 레이아웃을 잡는 것은 기한님께서 진행하실 예정
- (바울님 질문에 대한 답) 현재 사용하고 있는 데이터에서 implict branch, seq, isMergeCommit 프로퍼티는 아무도 사용하지 않음. taskId만 사용중.
