미션 목록 버그 수정

[HA-SEUNG-JEONG]

🌱 연관된 이슈

☘️ 작업 내용

#395 와 동일

🍀 참고사항

스크린샷 (선택)

Summary by CodeRabbit

릴리스 노트

• 버그 수정
  • 미션 카드의 상호작용 범위를 확대했습니다. 이제 일반 사용자가 제출 마감 상태의 미션 카드를 클릭할 수 있습니다. 리더의 카드 조작 방식은 변경되지 않았습니다.

[coderabbitai]

📝 Walkthrough

전체 요약

미션 카드 컴포넌트의 비리더 사용자를 위한 클릭 가능 여부 판별 로직을 확장하여, 기존 IN_PROGRESS 및 EVALUATION_COMPLETED 상태에 ENDED 상태를 추가했습니다.

변경 사항

코호트 / 파일	요약
미션 카드 클릭 로직 수정 src/components/card/mission-card.tsx	비리더 사용자가 미션 카드를 클릭할 수 있는 상태에 ENDED(제출 마감) 상태를 추가. 리더 경로는 변경되지 않음.

추정 코드 리뷰 노력

🎯 2 (단순) | ⏱️ ~10분

축하 시

🐰 미션의 끝도 선택의 시작,
마감 후에도 클릭할 수 있게,
모든 상태가 문이 되는 날,
누구나 그 길을 걸을 수 있네! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	PR 제목이 변경 사항의 핵심을 명확하게 설명하고 있습니다. '미션 목록 버그 수정'은 pull request의 주요 변화(미션 접근성 개선)를 간결하고 직관적으로 요약합니다.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/mission-list

---

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]

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

src/components/card/mission-card.tsx (1)

178-202: ⚠️ Potential issue | 🟡 Minor

비리더 관련 인라인 주석이 수정 사항을 반영하지 않음.

이번 변경으로 비리더 + ENDED 케이스가 클릭 가능 분기로 이동했지만, 해당 분기의 주석들은 아직 이전 동작을 서술하고 있습니다.

📝 주석 수정 제안

-  // 클릭 가능한 카드 (리더: 진행중/평가완료, 비리더: 진행중/평가완료)
+  // 클릭 가능한 카드 (리더: 진행중/평가완료, 비리더: 진행중/제출마감/평가완료)
   if (clickable) {

-  // 클릭 불가능한 카드 (비리더: 진행예정/제출마감)
+  // 클릭 불가능한 카드 (비리더: 진행예정)
   return (

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@src/components/card/mission-card.tsx` around lines 178 - 202, The inline
comment around the clickable branch no longer matches the code: the non-leader +
ENDED case was moved into the clickable branch, but the surrounding comments
still describe the previous behavior. Update the comments near the clickable
conditional in mission-card.tsx (the block using clickable, handleSelectMission
and rendering MissionCardContent) to accurately state that this branch now
covers leader: 진행중/평가완료 and non-leader: 진행중/평가완료 plus ENDED (or whatever the
ENDED status constant is), and adjust the subsequent comment before the
non-clickable branch to reflect the reduced/non-clickable cases; keep references
to clickable and handleSelectMission so reviewers can locate the change.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Outside diff comments:
In `@src/components/card/mission-card.tsx`:
- Around line 178-202: The inline comment around the clickable branch no longer
matches the code: the non-leader + ENDED case was moved into the clickable
branch, but the surrounding comments still describe the previous behavior.
Update the comments near the clickable conditional in mission-card.tsx (the
block using clickable, handleSelectMission and rendering MissionCardContent) to
accurately state that this branch now covers leader: 진행중/평가완료 and non-leader:
진행중/평가완료 plus ENDED (or whatever the ENDED status constant is), and adjust the
subsequent comment before the non-clickable branch to reflect the
reduced/non-clickable cases; keep references to clickable and
handleSelectMission so reviewers can locate the change.