Portfolio

기동 멘토님 피드백

CODE 관련

Input의 type 의 입력값이 조금 이상한것 같은데 조정이 필요한 부분이 아닐까 생각해봅니다.


<Input type="Email 인증 코드"

<Input type="기관명"

styled component 를 사용중이면서 styles도 혼용된 코드가 보입니다.


                <div
                  style={{
                    display: "grid",
                    justifyContent: "center",
                    alignItems: "center",
                    marginTop: "1rem",
                  }}
                >

BaseButton 등에서 fontSize, fontWeight 등의 props를 받고있던데 과연 적합한지 의문이 있습니다.



font의 사이즈에 따라서 lineHeight와 같은 부분들도 같이 변하게되고,
일정한 비율이 딱딱 맞아야 합니다.
이러한 부분들의 리스트업이 제대로 되어 있다면, 이에 맞춰서 리스트업을 하여 props로 fontSize를 받지 않고 
처리하는 것이 더 좋지 않을까 생각합니다.

propTypes으로 타입을 설정하여 사용한 부분 좋았던것 같습니다.

협업관련

git의 PR 규칙등은 잘 적용해서 prefix등을 잘 적용중인것 같네요. 근데 feat이랑 feature가 혼동되고 있는것 같은데, 사용자마다 조금씩 단어가 다르게 써서 그런것같네요.

전체적인 기능의 구현도나 남은 항목에 대해서 알 수 있어야 한다고 생각합니다. 칸반보드와 에픽등의 github 기능을 이용하여 수행할 수 있는 부분이었다고 생각이 듭니다. 척도로 명확하게 알 수 있어야 하는 것은 해당 프로젝트의 팀원이 개발자만으로 이루어진 것이 아닌, 다양한 사람들이 하나의 프로젝트 수행을 위해 달려나가고 있기 때문입니다. 앞으로 이러한 부분에 대해서 한번 더 고민해보고 전략을 수립해보시면 좋을 것 같다는 생각이 듭니다.

zenhub / jira등을 사용하는 이유이기도 합니다.

Github로 프로젝트 관리하기 - ZenHub 사용법

Github를 이용해서 Project Management 하는 방법 및 전체적인 프로세스에 대해서 정리했습니다. issue 관리, 일정 관리, 코드리뷰, 버그 리포트 등 다양한 일들을 Github 하나에서 다 관리할 수 있고 어느 하나 부족하다고 생각하지 않습니다. 이미 Remote Repository로 Github를 사용 중 이리 사면 적극 추천해 드립니다. 프로젝트 전체 코드는 GitHub에 공개 되어있습니다.

https://blog.dnd.ac/githhub-zenhub/

12조 중간 점검

기동 멘토님 피드백

CODE 관련

협업관련