[🐛BUG] QA

[Chasyuss]

🚀 풀 리퀘스트 제안

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

✈️ 관련 이슈

#332

📋 작업 내용

QA 수정

📸 스크린샷 (선택 사항)

📄 기타

Summary by CodeRabbit

• 스타일
  • 삭제 토스트 폭을 축소해 가독성과 일관성 개선.
  • 추천 할 일 헤더에서 ‘더 보기’ 액션 제거로 인터페이스 간소화.
• 버그 수정
  • 프로필 카드가 올바른 할 일 그룹(todoGroupId) 기준으로 다른 사용자 목록을 조회하도록 수정.
• 리팩터
  • 일부 데이터 조회에서 인증 토큰 의존성을 제거해 호출 방식 간소화 및 안정성 향상.
  • 커뮤니티 사이드바에서 상위 3개 항목만 표시하도록 제한.
• 품질 개선
  • 레벨 문자열 처리에 안전한 접근 방식 적용.

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

토스트 모달 너비값 조정, /v1/job/todo 호출에서 Authorization 헤더 제거, useJobOtherQuery 인자·엔드포인트를 jobId→todoGroupId로 변경 및 ProfileCard/호출부 업데이트, RecommendTodo에서 “More todo” 액션 제거가 포함됨.

Changes

Cohort / File(s)	Summary
ToastModal 너비 조정 src/common/CheckList.tsx	삭제 토스트 모달의 width 클래스가 w-[469px] → w-[350px]로 변경됨.
Job Todo 카테고리 조회 헤더 정리 src/hook/jobinfo/useJobTodoCategory.ts	localStorage에서 토큰 조회 및 Authorization 헤더 제거. /v1/job/todo GET을 쿼리파라미터만으로 호출하도록 단순화.
todoGroupId 전파 및 조회 엔드포인트 변경 src/hook/useJobQuery.ts, src/pages/jobDetail/components/ProfileCard.tsx, src/pages/otherTodoList/OtherTodoListPage.tsx	useJobOtherQuery의 인자가 jobId → todoGroupId로 변경되고, API가 /v1/todo/other/simple/${todoGroupId}로 수정됨. ProfileCard에 todoGroupId 필수 prop 추가 및 호출부 업데이트(OtherTodoListPage 포함).
RecommendTodo 헤더 UI 정리 src/pages/jobDetail/components/RecommendTodo.tsx	우측 정렬된 “More todo” 액션(텍스트+아이콘) 제거. 컴포넌트 시그니처 변화 없음.
Community 렌더링/정규화 소소 변경 src/pages/community/components/CommunityContents.tsx, src/pages/community/components/CommunityLeftSide.tsx	normalizeLevel에 optional chaining 적용(s?.replace), 인기 리스트 렌더링을 popularTodos.slice(0, 3)로 제한.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  participant PC as ProfileCard
  participant Q as useJobOtherQuery(todoGroupId)
  participant API as /v1/todo/other/simple/:todoGroupId

  PC->>Q: 호출(todoGroupId)
  Q->>API: GET /v1/todo/other/simple/{todoGroupId}
  API-->>Q: 200 OK (DreamerList)
  Q-->>PC: 데이터 반환

Loading

sequenceDiagram
  autonumber
  participant H as useJobTodoCategory
  participant API as /v1/job/todo

  H->>API: GET /v1/job/todo?id=&todoCategory= (헤더 없이)
  API-->>H: 200 OK (데이터)
  Note over H: try/catch 유지, 로깅/파싱 동일

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• main 최신화 #312: useJobOtherQuery 및 ProfileCard 관련 인자·엔드포인트 변경과 직접 중복되는 경로 수정 가능성.
• [✨FEATURE] 토스트 모달 UI 구현  #103: ToastModal 컴포넌트/프롭 변경과 관련된 모달 속성 수정 연관.
• [✨FEATURE] 커뮤니티 연결 #328: Community 컴포넌트(CommunityLeftSide/CommunityContents) 관련 UI 변경과 겹침.

Suggested labels

🐛Bug

Suggested reviewers

• cywin1018

Poem

깡총깡총, 모달 폭을 살짝 줄였네 🐇
todo 묶음엔 새 이름을 실어 보냈네
토큰은 잠시 벤치에 눕히고
화살표는 쉬도록, 한숨 돌리니
코드 밭에 햇살 한 줄기 반짝반짝

---

📜 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 8c9abaf and 1423b33.

📒 Files selected for processing (2)

• src/pages/community/components/CommunityContents.tsx (1 hunks)
• src/pages/community/components/CommunityLeftSide.tsx (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 bug/qa-332

---

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.}

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (3)

src/common/CheckList.tsx (1)

292-300: ToastModal 너비 축소 반영 OK. 다국어/줄바꿈 시 레이아웃 깨짐만 확인 부탁

긴 텍스트(예: 번역)에서 버튼 영역이 줄바꿈으로 밀리지 않는지 한번 확인해 주세요. 필요 시 반응형 클래스로 안전장치 권장.

가능한 대안:

-            width="w-[350px]"
+            width="max-w-[350px] w-auto md:w-[350px]"

src/hook/useJobQuery.ts (1)

170-177: 인증 헤더에 'Bearer null' 전송 가능성 차단 권장

토큰이 없을 때도 Authorization 헤더가 전송될 수 있습니다. 조건부 헤더로 방지해 주세요. 백엔드가 비로그인 접근을 허용하는지도 확인 바랍니다.

-  const token = localStorage.getItem('accessToken');
-  const response = await api.get(`/v1/todo/other/simple/${todoGroupId}`, {
-    headers: {
-      Authorization: `Bearer ${token}`,
-    },
-  });
+  const token = localStorage.getItem('accessToken');
+  const headers = token ? { Authorization: `Bearer ${token}` } : undefined;
+  const response = await api.get(`/v1/todo/other/simple/${todoGroupId}`, {
+    headers,
+  });

src/pages/jobDetail/components/ProfileCard.tsx (1)

16-19: 조회 기준 전환 확인 필요: 링크 이동은 여전히 jobId 사용

목록 데이터는 todoGroupId로 가져오지만 상단 "더 보러가기"는 /others/${jobId}로 이동합니다. 의도(직업 기준 페이지 이동)가 맞는지 확인 부탁드립니다. 만약 그룹 기준 이동이어야 하면 라우팅 파라미터도 정합성 맞춰주세요.

📜 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 694d0c1 and 8c9abaf.

📒 Files selected for processing (6)

• src/common/CheckList.tsx (1 hunks)
• src/hook/jobinfo/useJobTodoCategory.ts (0 hunks)
• src/hook/useJobQuery.ts (2 hunks)
• src/pages/jobDetail/components/ProfileCard.tsx (1 hunks)
• src/pages/jobDetail/components/RecommendTodo.tsx (0 hunks)
• src/pages/otherTodoList/OtherTodoListPage.tsx (1 hunks)

💤 Files with no reviewable changes (2)

• src/pages/jobDetail/components/RecommendTodo.tsx
• src/hook/jobinfo/useJobTodoCategory.ts

🔇 Additional comments (3)

src/hook/useJobQuery.ts (1)

185-189: todoGroupId 기반으로의 쿼리 전환 LGTM

쿼리 키와 파라미터 모두 일관성 있게 업데이트되어 캐시 충돌 우려 없습니다.

src/pages/otherTodoList/OtherTodoListPage.tsx (1)

111-116: ProfileCard에 todoGroupId 전달 변경 LGTM

데이터 모델(eachTodos.todoGroupId)과 컴포넌트 prop이 일치합니다.

src/pages/jobDetail/components/ProfileCard.tsx (1)

12-13: 새 prop(todoGroupId) 추가 반영 LGTM

타이핑과 사용처 일관성 문제없습니다.