Skip to content

Conversation

@cywin1018
Copy link
Contributor

@cywin1018 cywin1018 commented Sep 7, 2025

🚀 풀 리퀘스트 제안

  • 기능 추가
  • 기능 삭제
  • 버그 수정
  • 스타일링
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트
  • 기타

✈️ 관련 이슈

📋 작업 내용

📸 스크린샷 (선택 사항)

📄 기타

Summary by CodeRabbit

  • New Features

    • 채용 카드 클릭 시 상세 모달로 내용 확인 가능.
    • 커뮤니티·타인 할 일 카드에서 “내 할 일에 추가/취소” 지원 및 저장 수 표시, 토스트 알림 제공.
    • HOT 할 일 아이템 클릭 시 상세 목록으로 이동.
  • Style

    • 프로필 이미지 미제공 시 기본 이미지로 대체, 긴 텍스트 말줄임 처리.
    • 체크리스트 항목의 소형 추가 아이콘 제거.
    • 홈 “더 보러가기” 이동 경로를 커뮤니티로 변경.
    • 할 일 추가/취소 시 짧은 토스트 노출로 피드백 강화.

@cywin1018 cywin1018 requested a review from Chasyuss September 7, 2025 12:44
@coderabbitai
Copy link

coderabbitai bot commented Sep 7, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

여러 화면에서 할 일 추가 시 Amplitude 이벤트 트래킹을 도입하고(커뮤니티/직무 상세/타인 할 일), 커뮤니티 사이드바 클릭 이동 및 기본 프로필 이미지 처리, 리크루트 카드 클릭 시 모달 상세 보기 추가, 일부 타입 스키마 확장(todoGroupId, url, profileImage nullable), 체크리스트 UI의 AddIcon 요소 제거가 포함됩니다.

Changes

Cohort / File(s) Summary
Amplitude 트래킹 도입
src/utils/amplitude.ts
브라우저 환경에서 trackTodoImport(todoTitle) 추가. 경로 기반 source_page 추출 후 todo_import 이벤트 전송.
트래킹 적용(커뮤니티/직무 상세/타인 할 일)
src/pages/community/components/CommunityContents.tsx, src/pages/jobDetail/components/TabContent/*Contents.tsx, src/pages/otherTodoList/components/OtherTodoCard.tsx
할 일 추가 시 trackTodoImport 호출. 내부 핸들러 시그니처에 제목 인자 추가 및 클릭 핸들러 업데이트. 타인 할 일 카드에 저장/취소 토글, 카운트, 토스트 추가.
커뮤니티 사이드바 UX 개선
src/pages/community/components/CommunityLeftSide.tsx
리스트 행 클릭 시 이동(/otherslist/{id}), 기본 프로필 이미지 폴백, 텍스트 트렁케이션 및 스타일 조정.
홈 리크루트 모달 상세
src/pages/home/components/HomeRecruit.tsx
카드 클릭 시 모달로 상세 노출(CardDetail), 내부 타입 매핑/변환 추가, 좋아요 클릭 전파 차단.
홈 드리머 이동 경로 변경
src/pages/home/components/HomeDreamer.tsx
“더 보러가기” 이동 경로를 /jobfound/community로 변경.
API/타입 확장(홈/커뮤니티)
src/hook/useHomeQuery.ts, src/hook/community/query/useGetHotPopularQuery.ts, src/validation/home/popularSchema.ts
RecruitProps.url: string 추가, HotPopularItem.todoGroupId: number 추가, `PopularTodo.profileImage: string
체크리스트 UI 정리
src/common/CheckList.tsx
비편집 상태에서 AddIcon 블록 제거. 기타 로직 변경 없음.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor U as User
  participant C as UI (Community/JobDetail/Other)
  participant A as trackTodoImport
  participant M as Add/Delete Mutation
  participant T as Toast

  U->>C: "추가" 버튼 클릭
  alt 아직 추가되지 않음
    C->>A: trackTodoImport(title)
    A-->>C: (완료)
    C->>M: addTodoMutation
    M-->>C: 성공
    C->>T: "할일이 추가되었습니다." 표시(2s)
  else 이미 추가됨
    C->>M: deleteTodoMutation
    M-->>C: 성공
    C->>T: "할일이 취소되었습니다." 표시(2s)
  end
Loading
sequenceDiagram
  autonumber
  actor U as User
  participant H as HomeRecruit List
  participant D as Converter(convertToRecruitItem)
  participant MD as Modal(CardDetail)

  U->>H: 카드 클릭
  H->>D: 데이터 변환(RecruitData → RecruitItem)
  D-->>H: RecruitItem
  H->>MD: 모달 열기(item=RecruitItem)
  U-->>MD: 닫기 클릭
  MD-->>H: 모달 닫기
Loading

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~60 minutes

Possibly related PRs

Suggested labels

✨Feature, 💄Style

Suggested reviewers

  • Chasyuss

Poem

폴짝, 폴짝—커서가 춤추네
할 일을 담으면 별빛이 찍—툭!
토스트는 살짝 미소를 굽고
모달은 살며시 창을 펼치지
오늘의 흔적, 트래킹에 새기고
내일의 꿈은 리스트에 쏙! 🥕✨


📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b560dd6 and 1c227fe.

📒 Files selected for processing (12)
  • src/common/CheckList.tsx (1 hunks)
  • src/hook/community/query/useGetHotPopularQuery.ts (1 hunks)
  • src/hook/useHomeQuery.ts (1 hunks)
  • src/pages/community/components/CommunityContents.tsx (4 hunks)
  • src/pages/community/components/CommunityLeftSide.tsx (2 hunks)
  • src/pages/home/components/HomeDreamer.tsx (1 hunks)
  • src/pages/home/components/HomeRecruit.tsx (3 hunks)
  • src/pages/jobDetail/components/TabContent/SproutContent.tsx (3 hunks)
  • src/pages/jobDetail/components/TabContent/TreeContents.tsx (3 hunks)
  • src/pages/otherTodoList/components/OtherTodoCard.tsx (3 hunks)
  • src/utils/amplitude.ts (1 hunks)
  • src/validation/home/popularSchema.ts (1 hunks)
✨ Finishing Touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch develop

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@cywin1018 cywin1018 merged commit e029112 into main Sep 7, 2025
1 check was pending
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.

3 participants