-
Notifications
You must be signed in to change notification settings - Fork 7
3주차 멘토링
Q. 처음 로그인을 할 때 SWR이 undefined를 첨에 받는다 ㅠ 렌더링되기 전에 초기값을 설정할 수 있나
멘토님은 redux랑 useSWR을 쓰고 계신다.
SWR! 데이터가 넘어오기 전에는 undefined, 받아왔는데 데이터가 없을때는 빈 배열이 온다.
로딩 컴포넌트를 띄워주기. GraphQL에서도 로딩상태가 있어서 분기를 탄다.
라우팅할 때 새로고침하면 userData가 undefined라서 로그인창으로 튕겨저 나온다ㅠ (전역 데이터로 넣어서 로그인된 사용자인지 판별하고 싶었는데! 그래서 다른 방법을 사용 - 페이지를 이동할때마다 userData 요청을 한다 그래서 너무 요청이 많다고 생각)
→ 멘토님께서 코드를 더 읽어보시고 답변해주실 예정!
Q. 테스트 어떻게 어디까지 할까요??
소켓까지 단위테스트를 해가는건 비추! 시간이 빡빡하니까. 기본적인 렌더링이나 유틸성 테스트를 해보는 것이 좋을 것 같다.
e2e 테스트 나쁘지않다. 진입장벽이 있어서 테스트를 작성하는것 자체가 일이다. 테스트짜는 습관을 들이는 것은 좋다. 일정이나 리소스를 확보해둔 상태에서 하는 것이 좋겠다.
유틸성 단위테스트랑 e2e 테스트를 진행하고 있다.
Q. PR을 확인할 시간을 따로 갖나요?
pr을 리뷰하는 시간도 리소스를 갖는다.
시간이 엄청 길어질 것 같으면 merge하고 다음 pr에 리뷰를 반영한다.
여유가 없으면 오프라인 코드리뷰를 하거나 리팩터링하는 시간을 따로 마련한다
웬만하면 바로 리뷰를 상세하게 하는 것이 좋다 불필요한 시간을 줄일 수 있게
pr을 올리는 사람도 다른사람들이 이해하기 좋게 올려야 한다
유연함이 필요하다. 미리 approve를 한다든지 모아서 pr을 확인한다든지
하지만 모든 커밋이 history로 남기 때문에 잘 작성하는 것이 좋다
pr 단위, 커밋 단위도 중요하다
에러가 안나는 기준까지 pr을 날린다