-
Notifications
You must be signed in to change notification settings - Fork 1
9월 3, 4주차 회고(09.13 ~ 09.26)
Jungwon Lee edited this page Sep 25, 2021
·
2 revisions
- axios 관련 로직에 새로운 도전을 해봤다. interceptor을 사용하여 공통으로 사용될 부분들의 중복을 줄여보려고 했다.
- 코드 구현을 할 때 로직 중복이 생길 수 있는 부분을 미리 예상하고 문제가 발생하지 않도록 구현했다. OrderList 컴포넌트의 경우 기존 코드를 최대한 활용하도록 하였고 기능 확장에 어려움이 되는 요소를 제거하고, 추상화하여 코드의 재사용성을 높이는 방향으로 개선하였다.
- 팀원들의 코드를 다른 때보다 좀 더 유심히 보고 이해해보려고 했던 것 같다.
- PR이나 이슈 관리가 자세하게 잘 되어 있는 것도 있고, 다들 스케줄이 달라서 follow-up 하기가 쉽지 않은데 코드리뷰로 서로 진행상황 공유가 잘 되어서 다행이라는 생각이 든다.
- 팀원 리뷰를 제 시간에 해주지 못해서 너무 미안했다. 팀원이 바로 참고할 수 있도록 리뷰를 제 때 해둬야겠다.
- Login 상태 유지 로직을 미리 공부해두지 않고 또 미뤘었던게 역시 문제가 되었다. 링크만 던지지 말고 자세히 공부해야겠다.
- custom hook을 리팩토링 하려 했으나 하지 못한게 아쉬웠다.
- API 중 적용할 수 있는 기능이 몇몇 있던 것 같은데 이 부분을 백엔드 분들이랑 얘기를 나눠서 미리 적용해봤으면 좋았을 것 같다.
- 코드 리뷰 양이 적었던 것 같다.
- 과제구현과 면접으로 구현에 부진했던 것 같다. 팀원들한테 미안하고 고마운 생각이 든다. 아마 입사를 하게 되더라도 나머지 시간을 쪼개서 이어가야 할텐데 좀 더 효과적인 시간관리 전략이 필요할 것 같다.
- Oauth 기능 구현을 하는데 제니가 사전에 필요한 코드를 많이 준비해와서 회의가 수월하게 진행될 수 있었다.:) 분명 OAuth를 사용해본 적이 있는데 또 프로세스가 잘 생각이 나질 않는다. 해당 부분에 대한 온전한 이해가 필요할 것 같다. 다시 들여다 볼 것!
- 리액트 쿼리...학습할 방법만 고민하다가 또 시간이 갔다. 앞으로 사용될 부분에 대해서는 기존에 제니가 구현해주신 부분이랑 같이 200% 이해해볼 필요가 있을 것 같다.
- 코드리뷰를 미루지 않고 팀원의 코드가 pr에 올라오면 읽어보기
- 초기 렌더링 속도 개선방법 고민해보기
- API 관련 로직을 짤 때 로직 중복을 최대한 줄여보기
- 복잡한 로직이 추가되는 만큼 코드 리뷰를 적극적으로 해야겠다.
- 프로젝트 일정 관리 잘 하기
- 초기 렌더링 속도 개선하기 (최적화)
- 리팩토링 방향이 보이는 부분은 최대한 그때 그때 하자. 시간이 지나면 뭘 리팩토링 하기로 했는지도 까먹는 것 같다.
- 최적화 할 수 있는 방법들에 대해 고민해보기 (useCallback, useMemo 어떻게 사용하는지 공부해보기)
- 마감준수하기