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

feature: 디스코드 알림 설정 #31

Merged
merged 25 commits into from
Feb 13, 2025
Merged

Conversation

devmizz
Copy link
Contributor

@devmizz devmizz commented Feb 6, 2025

📌 Related Issue

🚨 해결하려는 문제가 무엇인가요?

  • 회원가입 신청 알림과 같이, 시스템 상에서 운영에 전달할 알림을 구현하기 위함입니다.

⭐️ 어떻게 해결했나요?

  • DIscord 채널 웹훅을 이용하여 알림을 전달합니다.

    • Discord에 Message를 전달하는 방식은 다양하나, App이나 다른 툴을 쓰기엔 배보다 배꼽이 커집니다.
    • 가장 간단하고 빠르게 구현 가능한 API 요청 방식을 활용했습니다.
  • 레이어가 살짝 복잡할 수 있습니다.

    • User 레이어에서는 Discord의 존재를 application과 domain이 알 수 없도록, infrastructure 레벨에서 DI를 통해 구현했습니다.
    • Discord도 추후 Client 구현 방식이 변경될 수 있지만, 가능성이 높지 않다고 판단하여 구현체로 두었습니다.
  • 추후 다른 PR이 머지되면 test, 개발, 상용 환경 변수 추가 필요합니다.

🤔 어떤 부분에 집중하여 리뷰해야 할까요?

🗒️ 참고자료

RCA 룰

  • R: 꼭 반영해 주세요. 적극적으로 고려해 주세요. (Request changes)
  • C: 웬만하면 반영해 주세요. (Comment)
  • A: 반영해도 좋고 넘어가도 좋습니다. 그냥 사소한 의견입니다. (Approve)

@devmizz devmizz added the 기능 New feature or request label Feb 6, 2025
@devmizz devmizz self-assigned this Feb 6, 2025
@devmizz devmizz force-pushed the feature/discord-notification branch from 11a784d to aebee0b Compare February 7, 2025 14:46
@devmizz devmizz merged commit 8445fe3 into develop Feb 13, 2025
1 check passed
@devmizz devmizz deleted the feature/discord-notification branch February 13, 2025 05:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
기능 New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant