Skip to content

[Fix] 주간/월간 캘린더 총 근무시간·예상 근무비 계산 오류 수정 (#55)#73

Merged
l2juhan merged 2 commits into
developfrom
bug/weekly-monthly-summary-calc#55
May 7, 2026
Merged

[Fix] 주간/월간 캘린더 총 근무시간·예상 근무비 계산 오류 수정 (#55)#73
l2juhan merged 2 commits into
developfrom
bug/weekly-monthly-summary-calc#55

Conversation

@l2juhan
Copy link
Copy Markdown
Collaborator

@l2juhan l2juhan commented May 4, 2026

📌 Issue number and Link

closed #55
#55

✏️ Summary

근로자 주간/월간 캘린더 상단 요약에서 총 근무시간·예상 근무비 값이 잘못 계산되던 버그 수정.

  • 총 근무시간: 모든 status(SCHEDULED/COMPLETED/DELETED) 시간을 합산하던 것을 → COMPLETED 근무만 합산으로 수정
  • 예상 근무비: 완료된 근무 급여만 표시되던 것을 → COMPLETED + SCHEDULED의 totalSalary 합산 (DELETED 제외)으로 수정
    • 백엔드 PR PayCheck-backend#177에서 SCHEDULED 근무도 totalSalary를 계산해 반환하도록 수정됨 (머지 완료) → 프론트에서 hourlyWage × 시간 추정 없이 백엔드 계산값을 그대로 사용

📝 Changes

  • src/utils/workSummary.ts (신규): calculateWorkSummary 유틸 함수. DELETED 제외, COMPLETED만 시간 합산, COMPLETED+SCHEDULED totalSalary 합산.
  • src/screens/worker/WorkerWeeklyCalendarScreen.tsx: 인라인 합산 로직 → calculateWorkSummary + useMemo로 교체
  • src/screens/worker/WorkerMonthlyCalendarScreen.tsx: 동일한 버그 패턴이 존재하여 함께 수정

🔎 PR Type

  • Feature
  • Bugfix
  • Code style update (formatting, local variables)
  • Refactoring
  • infrastructure related changes (CI/CD, Build)
  • Documentation content changes

📸 Screenshot

⚠️ 테스트 필요: 고용주 계정으로는 근로자 캘린더 화면을 확인할 수 없습니다. 근로자 계정 보유 팀원의 테스트가 필요합니다.

테스트 시나리오:

  1. 주간 캘린더 — 이번 주에 COMPLETED + SCHEDULED 근무가 모두 있을 때
    • 총 근무시간: COMPLETED 시간만 표시되는지
    • 예상 근무비: COMPLETED 급여 + SCHEDULED 급여(totalSalary) 합계가 표시되는지
  2. DELETED 근무가 있을 때 — 두 값 모두에서 제외되는지
  3. 근무가 없는 주/월 — 0시간, 0원 표시
  4. 위 시나리오를 월간 캘린더에서도 동일하게 확인

@l2juhan l2juhan self-assigned this May 4, 2026
@l2juhan l2juhan force-pushed the bug/weekly-monthly-summary-calc#55 branch from ded7b22 to 9b87852 Compare May 6, 2026 08:14
@l2juhan l2juhan requested a review from wonyeoe May 6, 2026 08:24
@l2juhan l2juhan merged commit 2b82571 into develop May 7, 2026
@l2juhan l2juhan deleted the bug/weekly-monthly-summary-calc#55 branch May 7, 2026 00:04
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.

[Bug] 주간 캘린더 '이번 주 총 근무시간'과 '이번 주 예상 근무비' 계산 오류

2 participants