Skip to content

fix: 기수 프로젝트 기반으로 학습 현황 집계 범위 수정#13

Merged
soobing merged 1 commit intomainfrom
fix/10-cohort-based-learning-status
Apr 8, 2026
Merged

fix: 기수 프로젝트 기반으로 학습 현황 집계 범위 수정#13
soobing merged 1 commit intomainfrom
fix/10-cohort-based-learning-status

Conversation

@soobing
Copy link
Copy Markdown
Contributor

@soobing soobing commented Apr 8, 2026

문제

재참여자(여러 기수 참여자)의 경우, 이전 기수에서 제출한 풀이가 현재 기수의 학습 현황에 포함되어 누적 풀이 수와 카테고리 진행도가 부정확하게 표시됨.

원인: fetchUserSolutions가 레포 전체 트리를 스캔하여 파일 존재 여부만 확인하므로, 기수와 무관하게 모든 풀이를 카운팅.

해결

열린 "리트코드 스터디X기" GitHub Projects를 기수 경계로 활용.

  • 현재 진행 중인 기수 프로젝트(closed: false) 자동 탐지
  • 해당 프로젝트에 연결된 유저의 머지된 PR만 대상으로 풀이 집계
  • 기수 프로젝트를 찾지 못할 경우 기존 전체 트리 스캔으로 폴백

변경 사항

  • utils/learningData.js: fetchCohortUserSolutions 함수 추가 (내부적으로 fetchActiveCohortProjectId, fetchUserMergedPRsInProject 사용)
  • handlers/learning-status.js: fetchUserSolutionsfetchCohortUserSolutions 교체

Closes #10

🤖 Generated with Claude Code

재참여자의 이전 기수 풀이가 현재 기수 누적 학습 현황에 포함되던 문제를
수정한다. 레포 전체 트리 스캔 대신, 열린 "리트코드 스터디X기" GitHub
프로젝트에 연결된 머지된 PR만을 기준으로 집계하도록 변경한다.

Closes #10

Co-Authored-By: sounmind <[email protected]>
Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
github 99cb542 Commit Preview URL

Branch Preview URL
Apr 08 2026, 12:50 PM

@soobing soobing merged commit 2a47983 into main Apr 8, 2026
1 check passed
soobing added a commit that referenced this pull request Apr 8, 2026
…rning-status"

This reverts commit 2a47983, reversing
changes made to 79358ee.
soobing added a commit that referenced this pull request Apr 8, 2026
revert: PR #13 (기수 프로젝트 기반 학습 현황 집계) 원복
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