Skip to content

#110 fix: 오답 제출 아이콘 미반영 문제 해결#110

Merged
JiiminHa merged 1 commit into
developfrom
fix/109-status
Mar 27, 2026
Merged

#110 fix: 오답 제출 아이콘 미반영 문제 해결#110
JiiminHa merged 1 commit into
developfrom
fix/109-status

Conversation

@JiiminHa
Copy link
Copy Markdown
Contributor

⚙️ Related ISSUE Number

close #109



📄 Work Description

오답 제출 아이콘 미반영 문제 수정합니다.



📷 Screenshot

스크린샷 2026-03-27 오전 10 34 26



💬 To Reviewers



🔗 Reference

@JiiminHa JiiminHa self-assigned this Mar 27, 2026
@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 27, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
snow-code-client Ready Ready Preview, Comment Mar 27, 2026 1:37am

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 27, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 3f59fe47-19ad-4b9b-a82b-cb7b79f8408b

📥 Commits

Reviewing files that changed from the base of the PR and between 504216a and ef7a9c8.

📒 Files selected for processing (1)
  • src/entities/student/ui/AssignmentProgressCard.tsx

📝 Walkthrough

Summary by CodeRabbit

새로운 기능

  • 과제 진행 상태 표시 개선
    • 제출됨 상태의 과제에 대해 "부정확함" 상태 표시 추가. 이제 올바름, 부정확함, 미제출의 세 가지 상태를 명확히 구분하여 사용자에게 더 정확한 피드백 제공

Walkthrough

AssignmentProgressCard 컴포넌트에서 오답 제출 시 표시할 Incorrect 상태 아이콘을 추가했습니다. getStatusIcon 함수의 조건문을 확장하여 isCorrect 값과 submittedCodeId 존재 여부에 따라 정답/오답/미제출 세 가지 상태의 아이콘을 적절히 렌더링하도록 개선했습니다.

Changes

Cohort / File(s) Summary
AssignmentProgressCard 아이콘 상태 처리
src/entities/student/ui/AssignmentProgressCard.tsx
Incorrect SVG 아이콘 import 추가 및 getStatusIcon 함수 로직 확장. 정답(isCorrect=true) → 오답(submittedCodeId 존재, isCorrect=false) → 미제출 순서의 3단계 조건문으로 변경. 함수 호출 시 assignment.submittedCodeId 전달.

📝 세부 검토 의견

긍정적인 부분:

  • 오답 제출 상태를 시각적으로 구분하는 버그 수정이 명확합니다. ✨

개선 고려사항:

  1. 조건문 우선순위 검증: 현재 로직에서 isCorrect 확인 후 submittedCodeId 체크하는 순서가 맞는지 확인해주세요.

    • isCorrect=false + submittedCodeId=null 시나리오가 발생 가능한지 검토하면 좋습니다.
  2. 테스트 커버리지: 각 아이콘 상태(정답/오답/미제출)별 테스트 케이스가 있는지 확인해주세요. 엣지 케이스 시나리오도 함께 검토하시면 더 견고한 코드가 됩니다.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/109-status

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.

@JiiminHa JiiminHa merged commit b5d3d88 into develop Mar 27, 2026
4 of 5 checks passed
@JiiminHa JiiminHa changed the title fix: 오답 제출 아이콘 미반영 문제 해결 #110 fix: 오답 제출 아이콘 미반영 문제 해결 Mar 27, 2026
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.

fix: 오답 제출 아이콘 미반영 문제 해결

1 participant