-
Notifications
You must be signed in to change notification settings - Fork 0
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
[기능구현] 보호자용 가이드라인 페이지 #105
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
100번째 이슈 축하합니다~~🎉
src/app/routes/index.tsx
Outdated
path: RouterPath.GUARD, | ||
element: <Layout title='가이드라인 목록' />, | ||
children: [ | ||
{ | ||
index: true, | ||
path: RouterPath.GUARD_GUIDELINE, | ||
element: <GuideLinePage />, | ||
}, | ||
], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이거 이런식으로 path 설정해도 보호자 페이지랑 보호자 가이드라인 페이지 둘 다 정상적으로 들어갈 수 있나요¿¿¿¿¿¿¿¿¿¿
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
잘 되던데 지금 보니 코드 상 좀 이상하네요..? 한 번 확인해볼게요!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수정했습니다 :)
{guidelineData?.map((guideline) => ( | ||
<GuideLineInfo | ||
key={guideline.Id} | ||
refetch={refetch} | ||
guideline={guideline} | ||
seniorId={Number(seniorId)} | ||
/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
이거 api가 있었군요! 몰랐넹...
type GuidelineInfo = { | ||
Id: number; | ||
type: string; | ||
title: string; | ||
content: string; | ||
}; | ||
|
||
const GuideLineInfo = ({ guideline }: { guideline: GuidelineType }) => { | ||
type Props = { | ||
guideline: GuidelineInfo; | ||
refetch: () => void; | ||
seniorId: number; // 수정 API 에 필요 | ||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Id
오타인가요?? Id
? id
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
저건 백엔드 스웨거에 Id 로 올라와있어서 일단 Id로 해놓았고 말씀드렸습니다!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
일단 이번 이슈에서는 이대로 처리하고, 이슈 새로 파서 수정해야할것 같네요
#️⃣ 연관된 이슈
📝 작업 내용
[기능구현] 보호자용 가이드라인 페이지 (가이드라인 추가, 수정, 조회)
가이드라인 조회, 추가, 수정 API 연동
차크라 UI 의 사용자 정의 props 에러
중간 커밋에 보면 $isMore 로 쓴 부분이 있었는데 이는 위에 스타일 컴포넌트의 props 관련 에러가 발생해서 gpt 에 물어보니 앞에 $ 를 붙여보라 해서 붙였었습니다 (엉터리 해결방법이었음.. 계속 warning 이 뜸). 왜 그런 지 찾아봤는데 요약하자면, Chakra UI 에 있는 Image 컴포넌트를 스타일 컴포넌트로 정의했을 때 용자 정의 props를 전달하려고 해서 발생하는 문제였습니다. 그래서 차크라UI 컴포넌트 말고 div 로 사용해서 해결했습니다.
Merge 전 할 사항
스크린샷 (선택)
시니어 선택하지 않고(null) 가이드라인 버튼을 눌렀을 때
시니어를 선택했을 때 (null -> 31)
가이드라인 페이지에 전달되어, 조회,수정,추가 등의 API 에 사용합니다.
💬 리뷰 요구사항(선택)
⏰ 현재 버그
✏ Git Close