Skip to content

Conversation

@yeonjin719
Copy link
Member

🚨 관련 이슈

✨ 변경사항

  • 🐞 BugFix Something isn't working
  • 💻 CrossBrowsing Browser compatibility
  • 🌏 Deploy Deploy
  • 🎨 Design Markup & styling
  • 📃 Docs Documentation writing and editing (README.md, etc.)
  • ✨ Feature Feature
  • 🔨 Refactor Code refactoring
  • ⚙️ Setting Development environment setup
  • ✅ Test Test related (storybook, jest, etc.)

✏️ 작업 내용

  • 안내문구 추가
  • 색상 통일
  • 버튼 통일
  • alert 추가

😅 미완성 작업

N/A

📢 논의 사항 및 참고 사항

@coderabbitai
Copy link

coderabbitai bot commented Aug 17, 2025

Caution

Review failed

The pull request is closed.

Summary by CodeRabbit

  • 신규 기능
    • 지역 검색 결과가 없을 때 빈 상태 메시지를 표시합니다.
    • 시작 시간이 비어 있을 경우 현재 시각으로 자동 설정됩니다.
  • 개선
    • 필터 초기화 시 안내 알림을 표시해 가시성을 높였습니다.
    • 페이지 내 이동 버튼을 일관된 스타일의 버튼으로 교체했습니다.
    • 코스 생성 실패 시 안내 문구와 간격을 개선하고 재시도 버튼/아이콘 호버 스타일을 추가했습니다.
    • 코스 결과 제출 시 장소가 없으면 제작 페이지로 이동하며 필터를 초기화합니다.

Walkthrough

  • 지역 검색 결과 렌더링 조건 변경 및 빈 상태 문구 추가
  • 필터 모달에서 startTime 미설정 시 기본값 자동 설정
  • 코스 페이지 버튼 컴포넌트로 교체 및 필터 초기화 시 alert 추가
  • 찾기 페이지 필터 초기화 시 alert 추가
  • 결과 페이지 제출 로직에 조건 분기 추가 및 useFilterStore에 reset 노출

Changes

Cohort / File(s) Summary
지역 검색 결과 빈 상태 처리
src/components/dateCourse/dateCourseSearchFilterOption.tsx
regionList 존재 시 항상 목록 컨테이너 렌더; regions 길이 0일 때 “검색 결과가 없습니다” 표시; 기존 매핑/클릭 로직 유지; 옵셔널 체이닝 도입.
필터 모달 startTime 초기화
src/components/modal/dateCourseSearchFilterModal.tsx
useEffect 추가: startTime이 없을 때 현재 날짜시간(YYYY-MM-DDTHH:MM)로 setField 호출.
코스 페이지 버튼/알림
src/pages/dateCourse/CoursePage.tsx
네비게이션 버튼을 Button 컴포넌트로 교체; “필터 초기화” 클릭 시 alert 후 reset 호출.
찾기 페이지 알림
src/pages/dateCourse/FindDateCourse.tsx
“필터 초기화” 클릭 시 alert 추가 후 reset 호출.
결과 페이지 제출 분기 및 안내
src/pages/dateCourse/MakeCourseResult.tsx
useFilterStore에서 reset 추가 분해; datePlaces 유무에 따라 mutate 수행 또는 /makeCourse로 이동+reset; 에러 안내 문구/마크업 및 버튼/아이콘 스타일 보강.
필터 스토어 API 확장
src/store/useFilterStore.ts
useFilterStore 반환 객체에 reset 공개(서명 확장).

Sequence Diagram(s)

sequenceDiagram
  participant U as User
  participant M as DateCourseSearchFilterModal
  participant F as Form State

  U->>M: 모달 열기
  M->>M: useEffect(startTime 검사)
  alt startTime 미설정
    M->>F: setField('startTime', 현재 시각 YYYY-MM-DDTHH:MM)
  end
Loading
sequenceDiagram
  participant U as User
  participant R as MakeCourseResult
  participant S as useFilterStore
  participant API as makeCourseMutate
  participant Nav as Router

  U->>R: 제출 클릭(handleSubmit)
  R->>R: datePlaces 존재/길이 확인
  alt datePlaces 있음
    R->>API: mutate(payload)
    API-->>R: onSuccess/onError
  else 없음
    R->>Nav: navigate('/makeCourse')
    R->>S: reset()
  end
Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

  • [Deploy] 배포 #128: useFilterStore에 reset 노출 및 관련 사용처 수정, 검색 옵션 컴포넌트 변경 등 동일 영역의 코드 수정과 직접적으로 연관.
  • [Bugfix] 에러 잡기 #127: dateCourseSearchFilterOption과 필터 초기화 흐름을 함께 다루며 본 PR과 동일 파일·로직 근접 변경.
  • [Deploy] 배포 #117: dateCourseSearchFilterModal에서 로딩/에러 처리 추가 등, 본 PR의 startTime 초기화 로직이 들어간 동일 모달에 대한 변경.

📜 Recent review details

Configuration used: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro

💡 Knowledge Base configuration:

  • MCP integration is disabled by default for public repositories
  • Jira integration is disabled by default for public repositories
  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 5e9029e and e2aba3e.

⛔ Files ignored due to path filters (1)
  • src/assets/icons/arrow_spin.svg is excluded by !**/*.svg
📒 Files selected for processing (5)
  • src/components/dateCourse/dateCourseSearchFilterOption.tsx (1 hunks)
  • src/components/modal/dateCourseSearchFilterModal.tsx (2 hunks)
  • src/pages/dateCourse/CoursePage.tsx (3 hunks)
  • src/pages/dateCourse/FindDateCourse.tsx (1 hunks)
  • src/pages/dateCourse/MakeCourseResult.tsx (2 hunks)
✨ Finishing Touches
  • 📝 Generate Docstrings

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

CodeRabbit Commands (Invoked using PR/Issue comments)

Type @coderabbitai help to get the list of available commands.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

Status, Documentation and Community

  • Visit our Status Page to check the current availability of CodeRabbit.
  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

@yeonjin719 yeonjin719 self-assigned this Aug 17, 2025
@yeonjin719 yeonjin719 merged commit ef99c37 into main Aug 17, 2025
2 of 3 checks passed
@yeonjin719 yeonjin719 deleted the refactor/#129 branch August 17, 2025 09:01
@yeonjin719 yeonjin719 changed the title feat: 사용성 높이기 [Refactor] 사용성 높이기 Aug 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants