Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: 구독한 구글폼 응답 최신화 및 알림 발송 #15

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

TaeyeonRoyce
Copy link
Contributor

@TaeyeonRoyce TaeyeonRoyce commented Feb 22, 2025

작업 내용

  • 구독한 구글폼을 주기적으로 조회하여 새로운 응답이 있으면 알림이 울리는 기능을 만들었습니다.

참고사항

코드 lint 하느라 변경된 파일들이 조금 있어요.
로직 변경 커밋 실제 로직은 위 커밋들입니다.

관련 이슈

PR 체크리스트

  • 테스트는 모두 통과했나요?
  • 빌드는 성공했나요?
  • 코드 포맷팅을 진행했나요?
  • PR 내부의 예시는 삭제하셨나요?

@TaeyeonRoyce TaeyeonRoyce force-pushed the 13-feat-구독한-구글폼-응답-최신화-및-알림-발송 branch from 8cab2c3 to d48b627 Compare February 24, 2025 14:36
@TaeyeonRoyce TaeyeonRoyce force-pushed the 13-feat-구독한-구글폼-응답-최신화-및-알림-발송 branch from d48b627 to 54a9551 Compare February 26, 2025 11:53
@TaeyeonRoyce TaeyeonRoyce marked this pull request as ready for review February 26, 2025 11:55
@TaeyeonRoyce TaeyeonRoyce self-assigned this Feb 26, 2025
@TaeyeonRoyce TaeyeonRoyce added the feature New feature development label Feb 26, 2025
@@ -2,5 +2,9 @@ spring:
application:
name: greedy-bot

main:
web-application-type: none
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이것은 웹 서버 없이 spring을 실행 하는 조건 같은데
스크린샷 2025-02-27 오후 12 13 55

  1. 해당 그림에서 tomcat 관련 기능들을 들고 오지 않는 것일까요?
  2. 저희 디스코드 앱은 controller없이 작동하기에 추가한 것일까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

네, 처음에도 web 의존서 없이 사용하다가 restClient를 위해 추가했었는데요.
DiscordBot은 소켓으로 연결해서 사용 되지 않는내장 톰캣을 띄우는게 비효율적이라 생각했습니다!

@davidolleh
Copy link
Collaborator

수고하셨습니다 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature development
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: 구독한 구글폼 응답 최신화 및 알림 발송
2 participants