Skip to content

feat: 제한 투표 참여자 소진 처리 - #9

Merged
Leesumok merged 2 commits into
mainfrom
feature/consume-restricted-participants
May 1, 2026
Merged

Leesumok merged 2 commits into
mainfrom
feature/consume-restricted-participants

Conversation

@Leesumok

@Leesumok Leesumok commented May 1, 2026

Copy link
Copy Markdown
Contributor

변경사항

  • 제한 투표에서 선택한 참여자가 투표를 완료하면 남은 참여자 목록에서 제거되도록 백엔드 저장 구조를 확장했습니다.
  • 방 조회 API는 제한 투표의 경우 아직 투표하지 않은 참여자만 내려주도록 정리했습니다.
  • 프론트 투표 화면은 남은 참여자 기준으로 선택 모달을 표시하고, 이미 소진된 참여자 충돌이 발생하면 방 정보를 다시 가져오도록 처리했습니다.
  • 참여자 소진 로직을 검증하는 백엔드 테스트를 추가했습니다.

배경

시크릿창이나 다른 브라우저에서 같은 투표 링크를 열었을 때 이미 투표한 참여자가 다시 선택 목록에 보이는 문제가 있었습니다. 기존에는 전체 참여자 목록만 저장하고 있어 투표 완료 후 특정 참여자를 소진 처리할 수 없었습니다.

검증

  • Backend: python -m pytest 통과
  • Frontend: npm ci 통과
  • Frontend: npm run lint 통과
  • Frontend: npx tsc --noEmit 통과
  • Frontend: npm run build 통과
  • Docker: backend/frontend 로컬 이미지 빌드 통과

참고

  • 로컬 uv가 없어 백엔드 테스트는 Python 3.12 테스트 venv로 실행했습니다.
  • 로컬 npm ci~/.npm 캐시 권한 문제를 피하려고 /private/tmp/fastvote-npm-cache 임시 캐시로 검증했습니다.

@Leesumok
Leesumok requested a review from dev-minsoo May 1, 2026 18:19
@Leesumok Leesumok self-assigned this May 1, 2026
@Leesumok
Leesumok marked this pull request as ready for review May 1, 2026 18:20

@dev-minsoo dev-minsoo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다.

@Leesumok
Leesumok merged commit 1f7b115 into main May 1, 2026
6 checks passed
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.

2 participants