Conversation
|
Caution Review failedThe pull request is closed. """ Walkthrough여러 UI 컴포넌트의 상호작용과 스타일이 개선되었습니다. Header 컴포넌트의 버튼 처리 로직이 확장되었고, EventRegisterLayout에 goHome 옵션이 추가되어 홈 네비게이션이 가능해졌습니다. 날짜 입력 UI와 초기 위치값, 디버깅 로그, CSS 포맷 등도 소폭 수정되었습니다. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant EventFunnel
participant EventRegisterLayout
participant Header
User->>EventFunnel: 이벤트 생성 시작
EventFunnel->>EventRegisterLayout: goHome={true} 전달
EventRegisterLayout->>Header: leftButtonLabel, leftButtonClick, rightContent 결정
User->>Header: 좌측 버튼 클릭
alt goHome이 true
Header->>User: 홈("/")으로 이동
else goHome이 false
Header->>EventRegisterLayout: onPrev() 호출
end
Possibly related PRs
Poem
""" 📜 Recent review detailsConfiguration used: CodeRabbit UI ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (7)
✨ Finishing Touches
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (5)
src/features/event/model/FunnelContext.tsx (1)
34-35: 기본 좌표값 하드코딩은 상수/환경변수로 분리 권장
locationLat,locationLng기본값을 서울시청 좌표로 직접 기입하면 테스트-환경 분리나 다국가 확장 시 유지보수가 어려워집니다.- locationLat: 37.5665, - locationLng: 126.9780, + locationLat: DEFAULT_LOCATION_LAT, // ex) 37.5665 + locationLng: DEFAULT_LOCATION_LNG, // ex) 126.9780상수를 별도 파일에 선언하거나
.env로 주입해 주세요.src/features/event/ui/EventType.tsx (1)
49-49: 디버그console.log는 제거하고 필요 시 로거로 대체하세요
console.log(eventState.locationLat, eventState.locationLng);는 개발 단계 디버그 용도로 보입니다. 운영 코드에 남아 있으면 콘솔 오염 및 성능 저하가 발생할 수 있습니다.- console.log(eventState.locationLat, eventState.locationLng); + // DEBUG: 위치 값 확인용 로그 — 필요 시 전용 로거(util/logger.ts) 사용커밋 단계에서 제거하거나
process.env.NODE_ENV !== 'production'조건부로 감싸는 것을 권장합니다.design-system/ui/Header.tsx (1)
44-48: 네이티브 버튼 외 커스텀 Button 컴포넌트는 걸러집니다
element.type === 'button'조건은<button>DOM 노드만 해당하므로forwardRef로 래핑된 버튼이나 디자인 시스템의IconButton컴포넌트는 통과하지 못합니다. 의도라면 주석으로 명시하고, 아니라면typeof element.type === 'function'등 추가 체크가 필요합니다.src/features/event/ui/EventFunnel.tsx (1)
84-85: 불린 props는 값 생략 권장JSX 불린 속성은 존재 자체가
true를 의미합니다.- goHome={true} + goHome불필요한
= {true}를 제거하면 코드 가독성이 향상됩니다.src/shared/ui/backgrounds/EventRegisterLayout.tsx (1)
50-57: 이중 네비게이션 호출 가능성 – 불필요한 중복 클릭 핸들러 제거 제안
leftButtonLabel안의IconButton과leftButtonClickprop 모두navigate('/')를 호출하고 있어,
같은 프레임에서 두 번 렌더링 / push 가 발생할 수 있습니다(react-router v6 는 동일 경로 push 시 무시하지만, side-effect hook 이 있으면 문제가 될 수 있음).불필요한 중복을 제거하여 의도를 명확히 하는 편이 좋습니다.
-leftButtonLabel={goHome ? ( - <IconButton - iconPath={<img src={HomeButton} />} - onClick={() => navigate('/')} - iconClassName="cursor-pointer z-30 ml-auto" - />) : '<'} -leftButtonClick={goHome ? () => navigate('/') : onPrev} +leftButtonLabel={ + goHome ? <IconButton iconPath={<img src={HomeButton} />} /> : '<' +} +leftButtonClick={goHome ? () => navigate('/') : onPrev}
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
yarn.lockis excluded by!**/yarn.lock,!**/*.lock
📒 Files selected for processing (7)
design-system/ui/Header.tsx(1 hunks)src/features/event/model/FunnelContext.tsx(1 hunks)src/features/event/ui/DatePicker.tsx(4 hunks)src/features/event/ui/EventFunnel.tsx(1 hunks)src/features/event/ui/EventType.tsx(1 hunks)src/index.css(1 hunks)src/shared/ui/backgrounds/EventRegisterLayout.tsx(3 hunks)
🧰 Additional context used
🪛 ESLint
design-system/ui/Header.tsx
[error] 19-19: Unexpected any. Specify a different type.
(@typescript-eslint/no-explicit-any)
⏰ Context from checks skipped due to timeout of 90000ms (1)
- GitHub Check: storybook
🔇 Additional comments (3)
src/index.css (1)
109-109: 불필요한 변경만 포함된 커밋마지막 개행 제거 외에 실질적 변화가 없습니다. 추적용 변경이 아니라면 해당 커밋은 생략하는 편이 이력 관리에 유리합니다.
src/features/event/ui/DatePicker.tsx (1)
140-144: 스타일 개선 👍
cursor-pointer추가로 클릭 가능함을 명확히 했고, 기존 로직에 부작용은 없습니다.
특별한 이슈 없이 머지 가능합니다.Also applies to: 189-193
src/shared/ui/backgrounds/EventRegisterLayout.tsx (1)
60-66: 조건부 렌더링 로직 명확하여 👍
goHome이 true 인 경우 우측 홈 버튼을 숨기는 처리와 className 유지가 잘 반영되었습니다.
xaexunxang
force-pushed
the
refact/#179/event-create-refactoring
branch
from
2ce2d3a to
063c0c0
Compare
2 participants
날짜 선택 중 날짜에 키보드 입력 제한 및 커서 비활성화
이벤트 오프라인 위치 좌표 설정 (서울 시청)
이벤트 등록 홈버튼 제거 및 첫 페이지에서 홈버튼만 렌더링
기타 사항 (노션 참고)
Summary by CodeRabbit