Skip to content

[BUG] 구글 캘린더 삭제 일정 동기화 누락 및 공휴일 중복 버그 수정 - #293

Merged
hoondongseo merged 1 commit into
developfrom
bug/simon/291-fix-calendar-sync
Aug 18, 2026
Merged

[BUG] 구글 캘린더 삭제 일정 동기화 누락 및 공휴일 중복 버그 수정#293
hoondongseo merged 1 commit into
developfrom
bug/simon/291-fix-calendar-sync

Conversation

@hoondongseo

Copy link
Copy Markdown
Contributor

🔗 이슈 번호

#️⃣ 기능 설명

  • 구글 캘린더에서 삭제(cancelled)된 일정이 트라이나 서버로 동기화되지 않아 유령 일정으로 남는 문제를 해결했습니다.
  • 다중 서버 환경(또는 스케줄러 동시 실행)에서 애플 공휴일이 DB에 2개씩 중복 생성되는 동시성 문제(Race Condition)를 원천 차단했습니다.

📌 작업 내용

  • GoogleCalendarClient.java: 증분 동기화(updatedMin) 호출 시 구글 API가 삭제된 일정을 응답에서 제외하지 않도록 timeMin, timeMax 파라미터를 쿼리에서 제거
  • V20__add_unique_index_for_holiday.sql: 기존 중복 공휴일 데이터를 초기화하고, 공휴일(HOLIDAY)의 external_event_id에 Unique Index를 생성하여 DB 레벨에서 중복 저장 완벽 방어 적용

✅ 체크리스트

  • Assignees, Labels를 모두 지정했나요?
  • GitHub Copilot의 자동 코드 리뷰 제안을 검토하고 필요한 부분을 반영했나요?
  • PR 머지 전 빌드 및 CI가 정상 작동하는지 확인했나요?

📸 스크린샷 (선택)

@hoondongseo hoondongseo self-assigned this Aug 18, 2026
@hoondongseo hoondongseo added the bug 버그 수정 label Aug 18, 2026
@coderabbitai

coderabbitai Bot commented Aug 18, 2026

Copy link
Copy Markdown

Important

Review available on request

  • 🔍 Trigger review

Reviews should be triggered manually for repositories with fewer than 10 stars. Select Trigger review above or comment @coderabbitai review to review the latest changes. For a full review, comment @coderabbitai full review.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 37a1bd8e-0b81-4a11-960f-8c9e5fc1c2b4


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.

@hoondongseo
hoondongseo merged commit 2d3bc57 into develop Aug 18, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug 버그 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant