View | 220831 Meeting Note

01 Discussion

1) 데이터 형식 맞추기

engine팀에 요청할 데이터 형식 맞추기

Q. view의 각 컴포넌트 팀이 모두 각각 처음부터 가공해서 시작하는 것 같은데, 그 형식 맞춰서 요청드리는 게 낫지 않을까요?

Q. 전역 상태 관리의 필요성

• 1 최상위에서 props로 내려주는 것

  • UI 구현이 목적이라면, data를 전역으로 가지고 있어야 할까요?
  • Temporal Filter의 가공된 내용을 위로 올려주는 방식은 어떨까요?

• 2 전역 상태 관리(context)사용

  • 모든 data를 context에 넣기에는 무겁다면, 필요한 index만 가지고 있는 건 어떨까요?
  • context에 모든 데이터를 가지고 있고, 컴포넌트마다 useMemo로 필요한 데이터를 골라 가지고 있는 건 어떨까요?

  => 1로 결정!

  => actions: (정혜)context 사용한 거 PR 올려주기

2) 개발 환경 문서

• https://github.com/githru/githru-vscode-ext/wiki

  => actions: 각 팀별로 wiki 작성해주기

3) PR

• 완성 되지 않아도 (껍데기라도 먼저) PR 날리기

• 완성되지 않았지만 논의하고 싶은 부분에 대해 PR로 Review 받기

• ex) simple version, class version

  => actions: 오늘(8/31) fake-asset 영진님이 PR올려주시면, => actions: 내일(이번주 토요일)까지 각 컴포넌트 PR 올려주기!!

4) Issue

자유롭게 활용하기! 추후 너무 더러워지면 Label 사용을 논의합시다!

5) 테스트

Q. 테스트를 어떻게 할까요?

• 1 jest를 사용해서 util 함수만 테스트?

• 2 UI 컴포넌트까지 테스트?

  • Storybook 사용에는 cost가 많이 들까요?
  • 컴포넌트 크기가 크기에 의미가 없을거 같다는 생각도 듭니다
  • 하지만 UI 테스트가 필요한 부분이 존재하긴 합니다

  => util은 테스트하고, 필요한 부분에 대해서만 UI 테스트를 적용!

  => actions: (기한님)UI 테스트를 해보고 PR 또는, Issue올려서 논의!

  => actions: 테스트가 필요한 분도 PR 또는, Issue올려서 논의!

6) 전체 환경 셋업

• husky를 사용한 lint-staged, merge CI 작업

  => actions: 하고 싶으신 분?

• 폴더 구조 📌

  VerticalClusterList 
   ㄴ index.ts
   ㄴ VerticalClusterList.tsx
   ㄴ VerticalClusterList.scss
   ㄴ VerticalClusterList.type.scss
   ㄴ Summary
       ㄴ index.ts
       ㄴ Summary.tsx
       ㄴ Summary.scss
       ㄴ Summary.util.ts
       ㄴ Summary.const.ts
       ㄴ Summary.type.ts
    ㄴ Graph
       ㄴ index.ts
       ㄴ Graph.tsx
       ㄴ Graph.scss
       ㄴ Graph.util.ts
       ㄴ Graph.const.ts
  

  => actions: 영진님이 PR 올려주기로!

7) fake-asset

=> actions: 영진님이 PR 올려주기로!

8) Detail Component에 어떤 내용이 들어갈껀지?

• 특정 작업자에 대한 커밋 정보를 모아서 보고 싶다는 수요(퇴사자 업무 파악용)
  • 어디서부터 언제까지 커밋을 만들었는지
  • 어떤 커밋키워드를 사용했는지
  • 어떤 파일을 주로 변경했는지

=> actions: 디테일 맡으신 분이 맡기로

9) scss 식별자 classname vs id

• PR에 해당 부분에 id를 사용하는 게 맞는 걸까 논의점 남겨주기

• svg에 그라데이션을 적용할 때, id를 많이 사용

• clip path에서 많이 사용

02 간식

• 오프라인 : 영택님이 스벅 배달해쥬심 ><
• 온라인 : 영택님이 개인DM 으로 기프티콘 보내쥬심 :)

03 일정

• 9/7 1차 마무리
• 9/3 토요일 회의 -> 투표

04 회의 사진

05 Next Action Items

• (정혜)context 사용한 거 PR 올려주기_
• 각 팀별로 wiki 작성해주기 (https://github.com/githru/githru-vscode-ext/wiki)
• 오늘(8/31) fake-asset 영진님이 PR올려주시면,
• 내일(이번주 토요일)까지 각 컴포넌트 PR 올려주기
• (기한님)UI 테스트를 해보고 PR 또는, Issue올려서 논의!
• 테스트가 필요한 분도 PR 또는, Issue올려서 논의!
• 전체 환경 셋업 하고싶은 분?