-
Notifications
You must be signed in to change notification settings - Fork 3
3주차 피어세션 그룹 36
ChiHyun Ahn edited this page Nov 13, 2020
·
1 revision
- 조별로 이번주 중점적으로 생각했던 사항 발표
- 발표가 끝나는 대로 약 30분간의 레포지토리 리뷰 시간을 가짐
- 리뷰를 진행하면서 질문 사항은 이곳에 등록
- 리뷰가 끝나고 질문에 대한 답변을 진행하는 시간
- 질문 양식 : 질문 - 질문자
- https://github.com/boostcamp-2020/IssueTracker-28
- 배포주소 : http://49.50.160.37:3000/
- PR 보내면서 결과물이 보여지는 것이면 캡쳐해서 PR 보낸것이 너무 좋네요(치현)
- 기술공유 위키 문서 너무 좋네요!!(치현)
- 페이지 이동시 보여지는 로딩 이미지는 어떻게 구현하셨나요?(치현)
- state와 dispatch를 다른 context로 관리하는 이유가 있나요? 장점?(치현)
- 이 페이지를 참고했습니다! (https://react.vlpt.us/mashup-todolist/02-manage-state.html )
- 에러페이지도 만드셨네요!! 굿굿!!(치현)
- (sidebar 폴더) useEffect(() => { }, [dispatch]); 이 로직이 하는 역할이 무엇인지 궁금합니다! (우민)
- (sidebar 폴더)trigger라는 로직이 컴포넌트 외부에 빠져있던데 하는 역할이 무엇인지 궁금합니다! (우민)
- constant로 고정적인 값을을 관리하신 점이 인상깊네요! (우민)
- 클라이언트 구조가 정말 신기한 것 같아요. 파일도 몇 개 없고 style로 일관되게 관리하신 점도 인상깊었습니다. 컴포넌트 단위가 크다고 생각해서 이부분이 아쉬웠는데, 꼼꼼히 보니까 딱딱 재활용이 가능한 단위의 크기로 나누신 것 같아서 기획서 분석을 정말 꼼꼼히 하신것 같다고 느꼈습니다. (우민)
- 데브서버에 프록시를 사용하셨네요. 그러면 배포시에는 api URL을 어떻게 관리하셨는지 궁금합니다. (우민)
- 로딩 페이지 처리해주신 거 참 좋았던 것 같습니다. (우민)
- https://github.com/boostcamp-2020/IssueTracker-38
- 배포주소 : http://27.96.130.13:3000/
- PR 작업단위와 충돌은 어떻게 해결하셨는지 궁금합니다 -동현
- 다른 유저의 작업이 실시간으로 반영되는게 너무 신기해요.. socket 부분 좀더 설명해주실 수 있을까요?! - 상경
- socket.io를 사용하신 이유가 궁금합니다! -동현
- server/app.js에서 dist폴더가 아닌 public/index.html로 하신 이유가 궁금합니다 -동현
res.sendFile(path.join(__dirname, 'public/index.html')); });
- Github Oauth를 passport 라이브러리 없이 직접 구현하신건가요 (설명도 듣고 싶습니다ㅎㅎ)👍👍 -동현
- custom hook 좋은 것 같아요! -동현
-
prop-types
라이브러리는 어떤 장점이 있을까요? -동현 - stores로 상태변화를 관리하는 부분이 인상깊어요! utils에 update, delete 모아놓으셔서 그런지 중복되는 부분 없이 깔끔해서 보기 좋아요ㅎㅎ -상경
- onClick 이벤트를 text보다 좀더 상위 컴포넌트(ex 드롭다운 item)에 걸면 클릭할때 좀더 편할것 같습니다😁 -상경
- CI/CD배포나 test에 대해서 고민하신점이 있으신지 궁급하니다 -동현
- 이슈 필터링하는 기능 로직 설명해주시면 감사하겠습니다🙇♀️ - 상경