Qa 348

[cywin1018]

🚀 풀 리퀘스트 제안

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

✈️ 관련 이슈

📋 작업 내용

📸 스크린샷 (선택 사항)

📄 기타

Summary by CodeRabbit

• New Features
  • 체크리스트에서 각 항목의 저장 횟수가 표시됩니다. 값이 없으면 0으로 표시됩니다.
  • 할 일 목록이 각 항목의 저장 횟수를 반영해 일관된 표시가 이뤄집니다.
  • 할 일 데이터가 저장 횟수와 저장 여부 정보를 포함하도록 업데이트되어 관련 정보 표시가 가능해졌습니다.

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

CheckList의 saveCount prop이 단일 숫자에서 항목별 배열로 변경되었고, 렌더링 시 배열 인덱싱을 사용합니다. Todo 컴포넌트는 CheckList에 saveCount 배열을 전달하고, useEffect 의존성이 location.pathname과 todoData로 확장되었습니다. Todo 스키마에 saveCount, isSaved가 nullable로 추가되었습니다.

Changes

Cohort / File(s)	Summary
CheckList 컴포넌트 API 변경 src/common/CheckList.tsx	CheckListProps.saveCount 타입을 number에서 (number | undefined)[]로 변경. 컴포넌트가 props에서 saveCount를 구조분해. 렌더링 시 saveCount[idx] ?? 0을 사용하여 항목별 저장 횟수 표시.
Todo 페이지 연동 및 트리거 변경 src/pages/myTodo/components/todo/Todo.tsx	useEffect 의존성을 location.pathname, todoData로 확장하고 콘솔 로깅 추가. GTM 이벤트가 해당 변경 시마다 실행. CheckList 호출 시 todoData.todos.map(t => t.saveCount ?? undefined)로 saveCount 배열 전달.
검증 스키마 확장 src/validation/mydream/todoSchema.ts	TodoSchema에 saveCount: z.number().nullable()와 isSaved: z.boolean().nullable() 추가. 내보내는 타입에 필드 확장.

Sequence Diagram(s)

sequenceDiagram
    autonumber
    actor User
    participant Todo as Todo 컴포넌트
    participant GTM as GTM
    participant CheckList as CheckList 컴포넌트

    User->>Todo: 페이지 접근 / 데이터 변경
    Todo->>Todo: useEffect(location.pathname, todoData)
    Todo->>GTM: 이벤트 디스패치
    Todo->>CheckList: props 전달(saveCount[]=todos.map(saveCount))
    CheckList->>CheckList: 항목별 saveCount[idx]로 렌더

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

당근 들고 뿅, 토깽이 출동해요 🥕
체크리스트는 칸칸이 저장수를 세어요.
투두는 길 바뀌면 살짝 신호 보내고,
GTM에 콩, 로그에 콩!
스키마도 두 칸 늘려, 꿈을 더 곱게 담아요.

---

📜 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 26ee433 and 54866e1.

📒 Files selected for processing (3)

• src/common/CheckList.tsx (3 hunks)
• src/pages/myTodo/components/todo/Todo.tsx (2 hunks)
• src/validation/mydream/todoSchema.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 qa-348

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[Chasyuss]

확인했어요