피어세션_w2 - boostcamp-2020/IssueTracker-28 GitHub Wiki

2주차 피어세션 그룹 21 🌞

진행 방식 🍎

  • 조별로 이번주 중점적으로 생각했던 사항 발표
  • 발표가 끝나는 대로 약 30분간의 레포지토리 리뷰 시간을 가짐
  • 리뷰를 진행하면서 질문 사항은 이곳에 등록
  • 리뷰가 끝나고 질문에 대한 답변을 진행하는 시간
  • 질문 양식 : 질문 - 질문자

공통 질문 👨‍🏫

  • userId를 localStorage에 저장해놓는게 괜찮을까? - 승현
  • allowImportExportEverywhere // dynamic require가 안좋은 이유? - 승현

Group 28 🎃

  • ✨⭐우주최강슈퍼킹왕짱조⭐✨ 로 이름을 짓게 된 계기가 있나요 - 승현
  • passport github strategy 이용하실 때, cors 에러 어떻게 해결하셨나요? - 승현
    • 28조 위키에 정리해놨습니다!
  • eslint, prettier 설정을 세세하게 하신 것이 인상적이네요. 처음부터 철저하게 계획하신 건지 궁금합니다 - 정훈
  • 파일구조가 깔끔하고 견고해보이네요! 레퍼런스가 있는지 궁금합니다. -정훈
    • 프론트는 벨로퍼트님의 velog 폴더구조를 많이 참고함ㅎㅎ
    • 백엔드는 한예지님의 가계부 폴더구조 참고 ㅎ
  • 저희도 비슷한 것 같은데, server 쪽과 client 쪽 import-export 방식이 다릅니다. commonJS 방식으로 할지 ES6+ 방식으로 할지 통일하는 것이 코드 일관성 측면에서 좀 더 좋을 것같아요 -정훈
  • sequelize에 mysql2 의존성이 있었나요? - 승현, 동훈
    • sequelize를 사용하려면 mysql2를 설치해야함
    • 따로 import는 안해줘도됨
  • styled-component ThemeConsumer 는 어떤 역할을 하나요? - 승현
  • css 도 따로 컨벤션을 적용하셨나요? - 승현
    • 네이밍 컨벤션만 적용
  • 공통으로 사용되는 input 등 tag를 별도로 만들어 놓고 사용하시는 부분이 좋아보이는데, 객체 속성을 따로 정하기 위해 props로 넣는 부분은 상속 등 다른 방법으로 할 수는 없을까요? 예를 들어 client/src/components/input/form/buttonWrapper.js 12, 19번 줄 같은 경우요! -정훈
  • sequelize를 사용하신 이유가 있나요? (장점이 궁금해요) - 동훈
    • 쿼리를 작성하지 않아도 된다
    • 가독성이 좋다
    • 테이블 자동 생성 -> 보기 편함
  • 야옹이가 너무 커엽습니다. -동훈
  • 회원가입할 때, 프로필사진은 어떻게 저장이되나요? - 승현
  • services/issue.js에 getIssues나 getComment... 등에서 결과값을 for문 돌리면서 다시 저장하는 이유가 있나요? - 민환
    • 관계 테이블 데이터를 가공하기 위해
  • 리펙토링은 어느 주기로 실행하시나요? - 승현
  • Github 프로는 어떤 장점이 있나요? - 승현
  • 보통 몇시까지 하시나요? - 민환
  • 기업 프로젝트 어떤거 선택하셨나요? -민환
  • 너무잘해서 질문할게 없어요.. - 익명
  • 저번 주에 비해 질문이 부드럽네요 -민환
  • 몇번정도 모이셨나요? -동훈
  • localStorage.getItem('user')은 어디서 저장된 데이터를 가져오는 건가요? -승현
  • babel 사용하는 경우에도 -webkit-progress-bar 이런 속성을 사용해야하나요? (client/src/components/milestones/List/Milestone/style.js 48번 줄 등) -정훈
  • context component를 사용하면 어떤 점이 좋을까요? - 승현
  • 를 최상단에 사용하고 switch path가 변경되면 다시 렌더링 되는건가요? - 승현
  • component 생성하는 함수는 function 을 사용하시고, 다른 대부분 경우는 화살표 함수를 사용하셨는데 구분하신 이유가 있나요? -정훈
  • milestone progress-bar css 를 -webkit-progress-bar 에서 적용하는게 맞나요? - 승현
  • client/src/utils/getFilterMessage.js에서 삼항연산자 여러개 사용되는 경우는 eslint 적용해서 좀더 가독성 좋게 해주시면 좋을 것 같아요 https://standardjs.com/rules-kokr.html 여기 삼항연산자 부분 참고해보시면 좋을 것 같습니다 - 정훈
  • 이슈나 프로젝트 관리는 누가하나요? 스크럼때 조정 하나요? - 승현
  • 코드 리뷰를 굉장히 활발하게 하셨네요 ㅎㅎ 멋집니다! -정훈

Group 30 💫

  • server/service 부분에서 req.body 부분을 구조 분해 할당으로 구현하시면 더 좋을것같아요! - 예진
  • client에서 이미지 생성한 코드가 멋집니다ㅎㅎ - 동현
  • 222... canvas를 사용하신 것 같은데 혹시 어느부분에서 사용하실 계획인지 궁금합니다! - 상경
  • 쿼리를 한곳에 모아놓고 사용한게 깔끔해보여요 - 예지
  • front에서 axios 사용하실때 url 저렇게 다 안적으시고 devServer에 프록시 설정에 http://localhost:3000 해주시면 바로 /user/signIn/auth 만 적어주셔도 요청갈거에요ㅎㅎ 
    proxy: {
  '/api/': 'http://localhost:3000',
}
    저희는 이렇게 했어요! - 예지 ---> 이거 찾아 헤맸는데 감사합니다.
  • detail 페이지에서 (/detail/:id) 새로고침을 해도 해당 page가 render가 잘 되는지 궁금합니다! - 동현
  • require('dotenv').config() 이부분은 제일 상단(index.js )에서 한번만 써주시면 전체 파일에서 env를 사용할 수 있어요ㅎㅎ - 상경
  • 배포버젼과 개발환경에서 환경변수(.env파일 등등..)를 다르게 관리하셨는지 궁금합니다 - 동현
  • front쪽 App.js가 길어보이는데 다른 함수로 분리해서 관리해도 좋을것같아요~! - 예지
  • 깃허브 아이콘은 octicons 라는 라이브러리를 사용하시면 이미지 파일을 저장하지 않으셔도 아이콘을 끌어다 사용하실 수 있어요 (https://primer.style/octicons/) - 상경
  • props를 아주 능수능란하게 사용하시는군요 멋져요 - 상경
  • sequelize를 사용하지 않으신 이유가 있나요!? - 예진
  • style 부분을 따로 빼는 것도 좋을것같아요! - 예진
  • 30조도 오프라인으로 모이시나요 ㅎ? - 예진
  • 30조는 분업이랑 짝프로그래밍 어떻게 하시는지 궁금합니다! - 예진
  • 벌금 얼마나 모였나여 ㅎㅅㅎ - 예진
  • 인터페이스 정의서 엑세스 막혀있어요ㅠㅠ - 예진 --> 풀었습니다!! --> 감사합니다!
  • 데모 url는 아직 없나요!? - 예진

3주차 피어세션 그룹 36 🌞

진행 방식 🍎

  • 조별로 이번주 중점적으로 생각했던 사항 발표
  • 발표가 끝나는 대로 약 30분간의 레포지토리 리뷰 시간을 가짐
  • 리뷰를 진행하면서 질문 사항은 이곳에 등록
  • 리뷰가 끝나고 질문에 대한 답변을 진행하는 시간
  • 질문 양식 : 질문 - 질문자

공통 질문 👨‍🏫

Group 28 🎃

  • https://github.com/boostcamp-2020/IssueTracker-28
  • 배포주소 : http://49.50.160.37:3000/
  • PR 보내면서 결과물이 보여지는 것이면 캡쳐해서 PR 보낸것이 너무 좋네요(치현)
  • 기술공유 위키 문서 너무 좋네요!!(치현)
  • 페이지 이동시 보여지는 로딩 이미지는 어떻게 구현하셨나요?(치현)
  • state와 dispatch를 다른 context로 관리하는 이유가 있나요? 장점?(치현)
  • 에러페이지도 만드셨네요!! 굿굿!!(치현)
  • (sidebar 폴더) useEffect(() => { }, [dispatch]); 이 로직이 하는 역할이 무엇인지 궁금합니다! (우민)
  • (sidebar 폴더)trigger라는 로직이 컴포넌트 외부에 빠져있던데 하는 역할이 무엇인지 궁금합니다! (우민)
  • constant로 고정적인 값을을 관리하신 점이 인상깊네요! (우민)
  • 클라이언트 구조가 정말 신기한 것 같아요. 파일도 몇 개 없고 style로 일관되게 관리하신 점도 인상깊었습니다. 컴포넌트 단위가 크다고 생각해서 이부분이 아쉬웠는데, 꼼꼼히 보니까 딱딱 재활용이 가능한 단위의 크기로 나누신 것 같아서 기획서 분석을 정말 꼼꼼히 하신것 같다고 느꼈습니다. (우민)
  • 데브서버에 프록시를 사용하셨네요. 그러면 배포시에는 api URL을 어떻게 관리하셨는지 궁금합니다. (우민)
  • 로딩 페이지 처리해주신 거 참 좋았던 것 같습니다. (우민)

Group 38 💫

  • 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에 대해서 고민하신점이 있으신지 궁급하니다 -동현
  • 이슈 필터링하는 기능 로직 설명해주시면 감사하겠습니다🙇‍♀️ - 상경