release/v1.3.2 #44
release/v1.3.2 #44
Conversation
- navigate(-1)이 실행되지 않던 문제 해결 - customBackHandler를 사용해 fromBookmarks 상태에 따라 분기 처리 - 일반적인 경우 false를 반환해 기본 navigate(-1)이 실행되도록 보장
- 더보기 클릭 시 제목의 줄임표 해제 - 내용 펼치기 상태에 따라 제목 표시 디자인 변경
- 여러 리뷰 카드를 렌더링하는 역할임을 명시 - 개별 리뷰 카드 컴포넌트 분리 후 export - return문에서 삼항연산자 제거
|
Caution Review failedThe pull request is closed. Walkthrough여러 헤더 컴포넌트와 PageLayout에 선택적 Changes
Estimated code review effort🎯 3 (보통) | ⏱️ ~20분
Possibly related PRs
Suggested labels
Suggested reviewers
Poem
Pre-merge checks and finishing touches❌ Failed checks (1 warning, 1 inconclusive)
✅ Passed checks (1 passed)
📜 Recent review detailsConfiguration used: Organization UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
Comment |
|
@Pinkippo 작업 완료 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
src/widgets/all-review-list/index.tsx (1)
83-89: 불필요한 타입 단언을 제거하세요.84번 라인의
as unknown as ReviewData[]타입 단언이 불필요합니다.reviews변수는 이미 63번 라인에서ReviewData[]로 올바르게 타입이 지정되어 있으며,ReviewCardListProps의review속성은ReviewData | ReviewData[]를 받을 수 있습니다.다음과 같이 수정하세요:
<ReviewCardList - review={reviews as unknown as ReviewData[]} + review={reviews} type={reviewType} fieldConfigs={fieldConfigs} showResource={true} limitItems={3} />
📜 Review details
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (11)
src/features/header/bookmark-header/index.tsx(2 hunks)src/features/header/community-header/index.tsx(3 hunks)src/features/header/kindergarten-header/index.tsx(3 hunks)src/features/header/save-header/index.tsx(2 hunks)src/pages/kindergarten/detail.tsx(2 hunks)src/shared/ui/layout/page-layout/index.tsx(4 hunks)src/widgets/all-review-list/index.tsx(2 hunks)src/widgets/community-feed/post-list/index.tsx(1 hunks)src/widgets/review-list/index.tsx(2 hunks)src/widgets/review-list/ui/ReviewCardList.tsx(3 hunks)src/widgets/review-list/ui/ReviewSummary.tsx(3 hunks)
🧰 Additional context used
🧬 Code graph analysis (4)
src/features/header/save-header/index.tsx (1)
src/features/header/base-header/index.tsx (1)
Header(49-139)
src/widgets/review-list/index.tsx (1)
src/widgets/review-list/ui/ReviewCardList.tsx (1)
ReviewCardList(29-60)
src/widgets/all-review-list/index.tsx (1)
src/widgets/review-list/ui/ReviewCardList.tsx (1)
ReviewCardList(29-60)
src/widgets/review-list/ui/ReviewCardList.tsx (1)
src/widgets/review-panel/lib/config.d.ts (1)
ReviewFieldConfig(1-4)
🔇 Additional comments (18)
src/widgets/community-feed/post-list/index.tsx (1)
13-14: 변경사항 확인 완료가상 스크롤 영역의 높이를 줄이기 위해 오프셋 값을 각각 10px씩 감소시켰습니다. PR 목표와 일치하며 로직상 문제가 없습니다.
src/widgets/review-list/ui/ReviewSummary.tsx (2)
13-13: prop 추가 확인 완료
isExpandedprop이 추가되어 리뷰 제목의 확장 상태를 제어할 수 있습니다.
50-55: 조건부 truncation 구현 확인 완료
isExpanded상태에 따라 제목에line-clamp-1클래스를 조건부로 적용하는 로직이 올바르게 구현되었습니다. PR 목표와 일치합니다.src/features/header/save-header/index.tsx (1)
11-11: 햄버거 버튼 제어 prop 추가 확인 완료
showHamburgerButtonprop이 추가되어 헤더의 햄버거 버튼 표시를 제어할 수 있습니다. 다른 헤더 컴포넌트들과 일관된 패턴을 따르고 있으며, 기본Header컴포넌트로 올바르게 전달됩니다.Also applies to: 20-20, 28-28
src/pages/kindergarten/detail.tsx (1)
20-33: 뒤로가기 네비게이션 개선 확인 완료
useUrlNavigation훅을 사용하여 북마크에서 진입한 경우와 일반 진입 경우를 구분하여 처리하는 로직이 올바르게 구현되었습니다. 이는 유치원 상세 페이지의 뒤로가기 동작 오류를 수정하는 PR 목표와 일치합니다.src/features/header/community-header/index.tsx (1)
16-16: 햄버거 버튼 제어 prop 추가 확인 완료다른 헤더 컴포넌트들과 동일한 패턴으로
showHamburgerButtonprop이 추가되고 올바르게 전달됩니다.Also applies to: 27-27, 43-43
src/features/header/bookmark-header/index.tsx (1)
10-10: 햄버거 버튼 제어 prop 추가 확인 완료일관된 패턴으로
showHamburgerButtonprop이 구현되었습니다.Also applies to: 18-18, 30-30
src/widgets/review-list/index.tsx (1)
7-7: 컴포넌트 리네이밍 확인 완료
ReviewCard에서ReviewCardList로 컴포넌트 이름이 변경되었으며, named export로 올바르게 import되고 사용됩니다. PR 목표에 명시된 리뷰 관련 컴포넌트 리네이밍이 적절하게 적용되었습니다.Also applies to: 49-49
src/features/header/kindergarten-header/index.tsx (1)
21-21: 햄버거 버튼 제어 prop 추가 확인 완료모든 헤더 컴포넌트에 일관되게 적용된
showHamburgerButtonprop 패턴이 올바르게 구현되었습니다. 이를 통해 비인증 페이지에서 사이드바 버튼 노출을 제어할 수 있습니다.Also applies to: 33-33, 48-48
src/widgets/all-review-list/index.tsx (1)
14-14: 컴포넌트 리네이밍이 올바르게 적용되었습니다.
ReviewCard에서ReviewCardList로의 변경이 컴포넌트 구조 개선과 일치합니다.src/shared/ui/layout/page-layout/index.tsx (4)
9-14: 인증 페이지 경로 관리가 적절합니다.중앙화된
URL_PATHS상수를 사용하여 인증 페이지를 정의한 것은 유지보수성을 높입니다.
39-39: 인터페이스 확장이 올바릅니다.옵셔널
showHamburgerButton속성 추가로 햄버거 버튼 표시를 명시적으로 제어할 수 있습니다.
89-92: 조건부 로직이 올바르게 구현되었습니다.명시적인
showHamburgerButton값이 우선하고, 미지정 시 인증 페이지에서는 햄버거 버튼을 숨기는 기본 동작이 적절합니다.
105-105: 헤더 컴포넌트로의 prop 전달이 적절합니다.계산된
shouldShowHamburgerButton값이 모든 헤더 컴포넌트에 올바르게 전달됩니다.src/widgets/review-list/ui/ReviewCardList.tsx (4)
21-27: 인터페이스 리네이밍과 타입 변경이 적절합니다.
ReviewCardListProps로의 이름 변경과 단일/배열 리뷰를 모두 수용하는 유니온 타입이 컴포넌트의 유연성을 향상시킵니다.
29-60: 리스트 컴포넌트 구현이 견고합니다.배열 정규화, 키 생성,
isLastItem계산이 모두 올바르게 구현되었습니다. 키 생성 로직은 리뷰 타입에 따라 적절한 ID 필드를 사용합니다.
64-64: 컴포넌트 내보내기 방식 변경과 코드 간소화가 적절합니다.명명된 내보내기로의 전환과 불필요한 템플릿 리터럴 제거가 코드 가독성을 향상시킵니다.
Also applies to: 124-124
135-135: 확장 상태를 ReviewSummary에 전달하는 것이 올바릅니다.
isExpanded상태를 전달함으로써 리뷰의 확장 여부에 따라 조건부로 제목을 잘라낼 수 있습니다. 이는 PR 목표인 "확장 상태에 따른 조건부 생략(말줄임) 적용"과 일치합니다.
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
src/pages/auth/find-password.tsx (1)
28-29: "재발급" 용어 사용 검토 권장사용자가 비밀번호를 찾는 과정에서 처음으로 임시 비밀번호를 발급받는 것이므로, "임시 비밀번호를 재발급"보다는 "임시 비밀번호를 발급"이 더 정확한 표현일 수 있습니다. 다만, 사용자 관점에서 기존 비밀번호를 대체하는 새 비밀번호를 받는다는 의미에서 "재발급"을 사용하셨다면 이해됩니다.
팀 내 용어 가이드라인이 있다면 이에 맞춰 확인해 주세요.
적용 가능한 수정안:
- {data.email as string}로 발송된 인증번호를 입력해주세요. <br /> - 인증 후 <strong>임시 비밀번호를 재발급</strong>해드릴게요. + {data.email as string}로 발송된 인증번호를 입력해주세요. <br /> + 인증 후 <strong>임시 비밀번호를 발급</strong>해드릴게요.
📜 Review details
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (2)
src/entities/auth/hooks/useResetPassword.tsx(1 hunks)src/pages/auth/find-password.tsx(1 hunks)
🧰 Additional context used
🧬 Code graph analysis (1)
src/entities/auth/hooks/useResetPassword.tsx (2)
src/pages/auth/sign-in.tsx (1)
SignInPage(26-64)src/pages/user/account-setting.tsx (1)
AccountSettingPage(11-69)
🔇 Additional comments (1)
src/entities/auth/hooks/useResetPassword.tsx (1)
23-29: LGTM! 사용자 경험 개선성공 메시지가 더 명확하고 친근하게 개선되었습니다. 본인 확인 완료 후 다음 단계를 구체적으로 안내하여 사용자 경험이 향상되었습니다.
이모지와 강조 표시를 통해 중요 정보를 효과적으로 전달하고 있으며, find-password.tsx의 메시지와도 일관성을 유지하고 있습니다.
⚒️ 작업 내용
⚙️ 관련 이슈
☎️ 관련 문의
Summary by CodeRabbit
New Features
Bug Fixes
Documentation
UI Copy
✏️ Tip: You can customize this high-level summary in your review settings.