Skip to content

[feat] Slack webhook 제거 및 Slack SDK 기반 전송#134

Merged
S2hyeyunS2 merged 10 commits intodevfrom
feat/#128
Jan 27, 2026
Merged

[feat] Slack webhook 제거 및 Slack SDK 기반 전송#134
S2hyeyunS2 merged 10 commits intodevfrom
feat/#128

Conversation

@S2hyeyunS2
Copy link
Collaborator

@S2hyeyunS2 S2hyeyunS2 commented Jan 23, 2026

📍 PR 타입 (하나 이상 선택)

  • 기능 추가
  • 버그 수정
  • 리팩토링
  • 기타 사소한 수정

❗️ 관련 이슈 링크

Close #128


📌 개요

  • Slack 전송 방식을 Webhook 기반에서 Slack 공식 SDK 방식으로 전환했습니다.
  • Slack 메시지 저장 이후 도메인 이벤트(AFTER_COMMIT)로 전송되도록 구조를 변경했습니다.
  • 이메일 기반 DM 전송(users.lookupByEmail → conversations.open → chat.postMessage) 확장 경로를 추가했습니다.

🔁 변경 사항

  • application/port/SlackClient 인터페이스 추가
  • SlackSdkConfig(+ botToken 설정) 추가 및 application.ymlslack.sdk.bot-token 설정 추가
  • SlackSdkClientAdapter 구현(채널 전송 / 이메일 DM 전송)
  • SlackMessageSavedEvent 도메인 이벤트 추가
  • SlackDomainEventHandler에서 AFTER_COMMIT 전송 및 상태(SENT/FAILED) 업데이트 처리
  • 기존 Webhook 관련 코드(SlackWebhookClient, WebClient config, DTO 등) 제거
  • notification 모듈 의존성 정리(WebFlux 제거, Slack SDK 추가)

📸 스크린샷

제목 스크린샷

👀 기타 더 이야기해볼 점


✅ 체크 리스트

  • PR 템플릿에 맞추어 작성했어요.
  • 변경 내용에 대한 테스트를 진행했어요.
  • 프로그램이 정상적으로 동작해요.
  • PR에 적절한 라벨을 선택했어요.
  • 불필요한 코드는 삭제했어요.

@S2hyeyunS2 S2hyeyunS2 changed the title [feat] [feat] Slack webhook 제거 및 Slack SDK 기반 전송 Jan 23, 2026
@S2hyeyunS2 S2hyeyunS2 self-assigned this Jan 23, 2026
@S2hyeyunS2 S2hyeyunS2 merged commit 62aec1c into dev Jan 27, 2026
2 checks passed
@S2hyeyunS2 S2hyeyunS2 deleted the feat/#128 branch January 27, 2026 06:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[refactor] Slack webhook 제거 및 Slack SDK 기반 전송 구조로 리팩토링

1 participant