-
Notifications
You must be signed in to change notification settings - Fork 1
[오피셜팀] 프론트엔드 코드리뷰용 PR #52
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: review
Are you sure you want to change the base?
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
…into feature/#127-applicate-mo
[DEPLOY] ci.yml 작성
…ithub.com/TEAMFICIAL/teamficial-fe into feature/#127-applicate-mo
[FEATURE] 지원하기 모달 반응형
[FEATURE] 키워드뷰 타인 진입 시 수정
[FEATURE] 모달창 수정
[오피셜팀] 팀피셜 프론트엔드 코드리뷰
✨ 리뷰를 요청드리는 주요 부분
가장 고민되는 지점을 아래 형식으로 정리했습니다.
src/components/modal/apply/profile/ProfileCard.tsxsrc/app/(main)/recruit/_components/profile/ProfileCard.tsxsrc/app/(main)/mypage/profile/_components/ProfileCard.tsxsrc/app/(main)/mypage/_components/profile/ProfileCard.tsxsrc/app/(main)/recruit/_components/*💬 오피셜팀이 고민하고 있는 부분
컴포넌트
ProfileCard.tsx파일이 각 다른 폴더에 4개가 중복되어있습니다.)src/app/(main)/project/[id]/_components/InfoCard.tsx&&src/app/(main)/project/[id]/applicants/_components/InfoCard.tsx두 컴포넌트는 UI가 동일하지만, 서버 응답 필드명이 달라 데이터 구조가 다릅니다. 통일 방법이 있을지 고민입니다.TipTap 에디터 관련
구조
src/app/(main)/project/[id]/applicants/page.tsx의 경로 depth 가 너무 깊어 파일트리 가독성이 떨어지는 것으로 느껴집니다. 현재 구조로 유지하는게 좋을지 기타 단위로 묶는 것이 좋을지 여쭤봅니다.💌 요청드리는 리뷰 방향
구조/설계 관점에서의 개선 제안을 주시면 많은 도움이 될 것 같습니다.
바쁘신 와중에 시간 내어 리뷰해주셔서 감사합니다.
편하게 피드백 부탁드립니다! 🥹