팀4 프론트엔드 최종 피드백

프로젝트 최종 점검

좋아요

1. 중간 점검 QA 때 발견한 버그 및 사용성 개선에 대한 이슈가 꽤 처리된 것처럼 보여요. 다들 최종 프로젝트 진행하느라 고생 많으셨습니다!

2. GitHub에서 프로젝트의주요 정보 및 페이지를 볼 수 있게 잘 정리해두신 것처럼 보여요.

개선 사항

1. package.json
package.json에는 name과 version 필드가 필수로 포함되어 있어야 합니다. 적절한 내용을 넣어주면 좋을 것 같아요.
https://docs.npmjs.com/creating-a-package-json-file#packagejson-fields

2. 디렉터리 구조를 적어두는 건 좋지만, 개별 파일을 모두 적어두는 것보다는 ‘구조'를 표현할 수 있도록 해주면 좋을 것 같아요.

소소한 버그

1. 로그인 페이지와 회원가입 페이지 간 컴포넌트 padding 불일치

2. 스페이스만 입력해도 중복 확인 진행



회원 가입을 할 때는 검증을 하고 있는데, 중복 확인을 진행할 때는 프론트와 서버 둘 다 검증을 하지 않는 것으로 보여요.

의견

1. 범용적으로 사용되는 공통 컴포넌트(common)를 styles/common.ts에 몰아 넣는 것도 유효하긴 하지만, 컴포넌트라는 관점에서 볼 때 공통 컴포넌트들도 components 쪽에서 관리하는 방식에 대해선 어떠신가요?

2. interface에 작성된 타입을 보아하니 거의 API에 관한 타입들처럼 보이는데요. 페이지에서 axiosInstance를 직접 import해서 사용하는 것보다는 api 자체를 서비스(모듈) 형태로 정의해서 제공하면 일일이 endpoint와 type을 매칭할 필요 없이 api 호출을 진행할 수 있지 않을까 싶어요~

3. 팀 소개에서 Product Owner와 Scrum Master라는 명칭의 역할의 책임을 명확히 이해하고 있나요? 그렇지 않다면 해당 표현은 삼가하는 게 좋을 것 같습니다. ⇒ 해당 R&R에 관한 질문을 받았을 때 잘못된 답변을 한다면 감점 요소가 되기 십상입니다. (노션에서 velopert라고 적혀있는 건 언어 유희… 겠죠…?)

조언

1. 현재 제가 판단할 수 있는 부분에 대해서 최종 점검을 진행하기는 하지만, 프로젝트에 관한 이해도가 가장 높은 건 이 프로젝트를 끝까지 이끌어 온 여러분들입니다. 그러므로 프로젝트의 특징, 장점, 본인의 역할을 등 프로젝트를 가장 잘 이해하고, 표현할 수 있는 것도 여러분들입니다. 그러므로 스스로에게 자신감을 가지고 (리팩터링도 하고) 이력서 작성이나 인터뷰를 진행할 때 여러분이 기여한 내용을 가감 없이 보여줄 수 있기를 바라요.

2. 이번 팀 프로젝트를 진행하면서 겪은 긍정적인 경험은 앞으로도 쭉 이어나가시길 바라고, 부정적인 경험은 되풀이 하지 않도록 하면서 계속 이어나갈 수 있기를 바랍니다.