Skip to content

fix: 새로 올린 식단의 Slack 알림 복구, 도담 휴무 문장 처리 - #40

Merged
rover0811 merged 2 commits into
mainfrom
fix/restore-slack-summaries
Oct 4, 2026
Merged

rover0811 merged 2 commits into
mainfrom
fix/restore-slack-summaries

Conversation

@rover0811

@rover0811 rover0811 commented Oct 4, 2026 •

Copy link
Copy Markdown
Collaborator

요약

식단을 새로 올리면 Slack 알림을 다시 보냅니다. 도담 휴무 공지도 휴무로 처리합니다.

리뷰어가 할 일

  • functions/handler.py의 newly_published_dates를 확인

prod에는 이미 배포 완료

무엇이 고장났나요?

  1. PR fix: 빈 식단 칸이 없어질 때까지 재확인하고 중복 업로드 방지 #39 이후 Slack 알림이 오지 않았습니다.
  2. 도담이 휴무일에 재시도를 반복했습니다.

무엇을 바꿨나요?

  1. 새로 올린 날짜만 Slack으로 알립니다. 이미 있는 날짜와 휴무일은 알리지 않습니다. 그래서 하루에 여러 번 실행해도 같은 알림이 반복되지 않습니다.
  2. 모든 스케줄에서 notify_summary를 true로 바꿨습니다.
  3. "휴무 입니다"로 끝나는 문장을 휴무로 처리합니다.
자세한 내용 (필요할 때만 읽으세요)

알림 규칙

상황 Slack
새 식단을 올림 그 날짜의 메뉴를 알림
식단이 이미 있음 알리지 않음
휴무일 알리지 않음
기숙사 10:00에 오늘 칸이 비어 있음 경고 (변경 없음)
일요일 재시도가 모두 실패함 경고 (변경 없음)
수동 실행 이전처럼 알림

10/4 조치

  • 10/5부터 10/11까지 도담, 학생식당, 교직원식당 식단은 모두 올라가 있습니다. 중복은 없습니다.
  • 빠진 Slack 알림 16건은 수동으로 보냈습니다.
  • 10/5(대체공휴일)와 10/9(한글날)는 휴무입니다.
  • 배포 후 도담 실행 결과는 12칸 중 12칸 확보입니다.

테스트

전체 201개가 통과합니다. 추가한 테스트는 다음과 같습니다.

  • 새 날짜가 하나 있으면 알림도 하나만 나가는 경우
  • 모든 날짜가 이미 있으면 알림이 나가지 않는 경우
  • 도담 휴무 문장이 휴무로 처리되고, 일반 메뉴는 메뉴로 처리되는 경우

배포 규칙 (PR #39와 같음)

  • rm -rf .aws-sam 후 빌드합니다.
  • ReservedConcurrentExecutions를 추가하지 않습니다.

@rover0811
rover0811 merged commit a6b0de3 into main Oct 4, 2026
1 check 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.

1 participant