[조원정] sprint 9#145
Merged
gyulrangdev merged 1 commit intocodeit-sprint-fullstack:next-조원정from Oct 7, 2025
Hidden character warning
The head ref may contain hidden characters: "next-\uc870\uc6d0\uc815-sprint9"
Merged
Conversation
gyulrangdev
reviewed
Oct 7, 2025
Collaborator
gyulrangdev
left a comment
There was a problem hiding this comment.
구조도 체계적이고 컴포넌트 분리도 잘 되어있네요.
에러 처리를 공통화하고, JsDoc 으로 주석 추가하는 방법도 알아보시면 좋겠습니다. (필수는 아닙니다)
https://poiemaweb.com/jsdoc-type-hint
고생많으셨습니다~
| @@ -0,0 +1,70 @@ | |||
| const URL = `${process.env.NEXT_PUBLIC_CODEIT_URL}`; | |||
Collaborator
There was a problem hiding this comment.
제 환경에서는 env 파일이 없어서 http://localhost:3000/undefined/auth/signIn 경로로 요청이 가서 정상적인 확인이 어렵네요 ㅠ
env 파일 자체는 레포에 올리지 않는 것은 맞습니다. 따라서 실무에서는 env.example 형태로 예시 파일을 추가하고 실제 env 정보는 DM으로 전달하거나 따로 문서로 관리하는 편입니다.
참고해주시면 좋겠습니다~
| }; | ||
|
|
||
| // product get | ||
| export const fetchProduct = async ({ id }) => { |
Collaborator
There was a problem hiding this comment.
용도별로 fetch를 따로 나눈 점이 좋습니다 👍
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
요구사항
기본
공통
로그인/회원가입 페이지
로그인 페이지
회원가입 페이지
로그인, 회원가입 페이지 공통
GNB
상품 상세 페이지
심화 요구사항
로그인 및 회원가입 페이지 공통
PC: 1200px 이상
Tablet: 744px 이상 ~ 1199px 이하
Mobile: 375px 이상 ~ 743px 이하
375px 미만 사이즈의 디자인은 고려하지 않습니다
유저 기능
리퀘스트 헤더에 인증 토큰을 첨부할 때 axios interceptors를 활용해 주세요. (axios를 사용하지 않는다면 이와 유사한 기능을 활용해 주세요.)
React-Query로 마이그레이션
fetch 혹은 axios로 구현된 기존의 API 요청 코드를 React-Qeury로 마이그레이션 합니다.
로딩 및 에러 핸들링
상품 데이터 캐싱 및 업데이트
주요 변경사항
멘토에게