Skip to content

Fix/dashboard ongoing start at - #436

Merged
kimjiy9607 merged 3 commits into
betafrom
fix/dashboard-ongoing-startAt
Nov 8, 2026
Merged

kimjiy9607 merged 3 commits into
betafrom
fix/dashboard-ongoing-startAt

Conversation

@kimjiy9607

Copy link
Copy Markdown
Contributor

요약

변경 사항

체크리스트

Self Review

  • 내 코드를 직접 리뷰했다 (/self-review 실행 또는 수동)
  • 불필요한 console.log / println / 디버그 코드를 제거했다
  • 변수·함수 이름이 의도를 명확히 드러낸다
  • 에러 핸들링이 빠진 경로가 없다

테스트

  • Stage 에서 테스트했다 (해당 시 스크린샷 첨부)
  • 단위/통합 테스트를 추가 또는 업데이트했다
  • 기존 테스트가 깨지지 않는다 (./gradlew test / pnpm test)

보안 (PUBLIC 레포)

  • 시크릿·토큰·API 키·PII 가 코드에 없다
  • .env 파일이 커밋에 포함되지 않았다

기타

  • 스키마 변경 시 V{n}__*.sql 마이그레이션을 같이 넣었다
  • 관련 문서(ERD, 스펙)를 업데이트했다
  • 01-planning/stories/** 또는 frontend/apps/playground/**/spec.ts 를 건드렸다면 pnpm --filter playground export-specs --check 통과 확인

팀 공유

  • 팀이 알아야 할 결정이 생겼다 → docs/share/ 에 한 장 (없으면 지운다)
  • prod 사고 수정이다 → docs/incidents/ 에 한 장 (없으면 지운다)

Jiyoon Kim and others added 2 commits November 7, 2026 16:28
ONGOING 상태로 전환하지 않기로 한 결정 5(study-meeting spec)에 맞게
STATUS=ONGOING 판정을 STATUS=OPEN+START_AT<=now 로 교체한다.

- StudyParticipantRepository: findActiveInOngoingStudies → findActiveInStartedOpenStudies(Instant asOf)
- BackofficeStudyListFilter: startedOnly 필드 추가 — DAO에서 START_AT 조건 적용
- AdminStatsService.ongoingBoard: status=OPEN + startedOnly=true
- AdminStatsService.studiesByCategory: 진행 중 카운트에 OPEN+startAt 경과 포함
- DashboardSummaryAggregateService: 변경된 메서드 호출로 업데이트
- AdminStatsIntegrationTest: 시드를 OPEN+과거 START_AT 으로 변경, 보드 테스트 조건 명확화

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
회차 스펙 결정 5(OPEN→ONGOING 전환 없음)에 맞게 dashboard/spec.md 의
'진행 중 기수' 정의와 집계 조건을 STATUS=ONGOING → STATUS=OPEN+START_AT 경과로 수정한다.

- 용어 표, 2-1절 판정 근거, 3-1절 보드 조건, 6절 카테고리 범위,
  4.3절 categories[].ongoingCount 설명, 5절 검증 기준, FE 주소-쿼리 연결 표 7곳 갱신
- 결정 5로 두 보드 모두 OPEN 대상이 돼 동시 포함 가능성 명시

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@kimjiy9607
kimjiy9607 requested a review from titaniper as a code owner November 8, 2026 00:38
@kimjiy9607
kimjiy9607 merged commit a6f4937 into beta Nov 8, 2026
3 checks passed
@kimjiy9607
kimjiy9607 deleted the fix/dashboard-ongoing-startAt branch November 8, 2026 00:45
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.

1 participant