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: 댓글 신고 누적 시 슬랙에 알림 보낸다 #198

Merged
merged 1 commit into from
Mar 16, 2024
Merged

Conversation

melonturtle
Copy link
Contributor

What is this PR? 🔍

  • 토픽 신고 누적 시엔 알림 보냈는데, 댓글 신고 누적 시엔 알림 보내는 게 없어서 추가했습니다.
    • 그 전엔 임시로 만들었었는데 웹에서 정식 스펙으로 결정되었으니 추가했습니다.

🛠️ Issue

Changes 📝

  • 기존에 ReportHandler도 토픽 신고만 메시지로 보냈는데 각 신고 이벤트에 맞게 메시지 보내도록 ReportType enum 추가해서 활용했습니다.
    • 만약 나중에 슬랙에 다른 알림도 보내고 싶으면 ReportEventHandler를 추상 공통 핸들러로 만들고 메세지 만드는 것만 파생 클래스에서 정의하는 식으로 해도 될듯??

기존에 있던 report handler도 report 추가하기 쉬운 구조로 변경한다
@melonturtle melonturtle requested a review from 60jong March 16, 2024 05:50
@melonturtle
Copy link
Contributor Author

요거 출시 전에 머지되는 게 나을 것 같아서 지금 머지하고 테스트 해볼게요

@melonturtle melonturtle merged commit 8bfaf0c into main Mar 16, 2024
1 check passed
@melonturtle melonturtle deleted the report branch March 16, 2024 05:51
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