Skip to content

[FIX/#123] 투표 현황 - 총 투표자 수 수정 - #124

Merged
vahkjsdf merged 1 commit into
developfrom
fix/#123-total_vote_count
Sep 28, 2026
Merged

vahkjsdf merged 1 commit into
developfrom
fix/#123-total_vote_count

Conversation

@vahkjsdf

@vahkjsdf vahkjsdf commented Sep 20, 2026 •

Copy link
Copy Markdown
Contributor

Related issue 🛠

Work Description ✏️

  • 투표 현황 - 총 투표자 수 수정했습니다.

Screenshot 📸

  • N/A

Uncompleted Tasks 😅

  • N/A

Summary by CodeRabbit

  • 버그 수정
    • 투표 현황에서 특정 후보의 득표 수가 아닌 전체 투표자 수가 표시되도록 수정했습니다.
    • “투표했어요” 안내 문구의 숫자가 전체 투표 참여자를 정확히 반영합니다.

@vahkjsdf vahkjsdf self-assigned this Sep 20, 2026
@vahkjsdf vahkjsdf added the 🔨 FIX 버그 수정 label Sep 20, 2026
@coderabbitai

coderabbitai Bot commented Sep 20, 2026 •

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: moive-app/moive-android/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 88ebbd95-423f-4c67-b9ba-80b92fc93876

📥 Commits

Reviewing files that changed from the base of the PR and between cdd2957 and a77f056.

📒 Files selected for processing (2)
  • app/src/main/java/com/moive/app/presentation/votestatus/component/PlaceVoteSection.kt
  • app/src/main/java/com/moive/app/presentation/votestatus/component/ScheduleVoteSection.kt

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

투표 현황 화면 두 곳에서 VoterCountCaption이 선두 후보의 투표 수가 아닌 전체 투표자 수를 표시하도록 변경했습니다.

Changes

전체 투표자 수 표시

Layer / File(s) Summary
VoterCountCaption 값 변경
app/src/main/java/com/moive/app/presentation/votestatus/component/PlaceVoteSection.kt, app/src/main/java/com/moive/app/presentation/votestatus/component/ScheduleVoteSection.kt
VoterCountCaption의 voterCount 인자를 topVoterCount에서 totalVoterCount로 변경했습니다.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Bug fix

🚥 Pre-merge checks | ✅ 4 | ❌ 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%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 2 files. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 투표 현황에서 총 투표자 수를 수정하는 주요 변경 사항을 정확하고 간결하게 설명합니다.
Description check ✅ Passed PR 설명이 템플릿의 필수 섹션을 포함합니다. 관련 이슈 #123, 작업 내용, 스크린샷 및 미완료 작업 상태를 명시했습니다.
Linked Issues check ✅ Passed 이슈 #123의 코딩 요구사항은 투표 현황의 총 투표자 수에 total_vote_count를 반영하는 것입니다. PlaceVoteSection.kt와 ScheduleVoteSection.kt는 VoterCountCaption에 topVoterCount 대신 totalVoterCount를 전달합니다. 따라서 두 표시 경로가 총 투표자 수…
Out of Scope Changes check ✅ Passed 변경 범위는 이슈 #123의 투표자 수 표시 수정에 한정됩니다. 두 컴포넌트의 VoterCountCaption 인자만 변경되었으며, totalVoterCount 반영과 직접 연결됩니다. 관련 없는 변경은 확인되지 않습니다.
  • Fix all pre-merge checks with AI
✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

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.

@vahkjsdf
vahkjsdf merged commit 7e73c96 into develop Sep 28, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🔨 FIX 버그 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FIX] 투표 현황 - 총 투표자 수 수정

1 participant