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] 슬랙 알람 연동 #156

Merged
merged 6 commits into from
Dec 19, 2023
Merged

[FEAT] 슬랙 알람 연동 #156

merged 6 commits into from
Dec 19, 2023

Conversation

yujindonut
Copy link
Member

- commit message가 적절한지 확인해주세요.
- 마지막으로 Coding Convention을 준수했는지 확인해주세요.
- 적절한 branch로 요청했는지 확인해주세요.
- Assignees, Label을 붙여주세요.
- 가능한 이슈를 Link 해주세요.
- PR이 승인된 경우 해당 브랜치는 삭제 부탁드립니다.

✨ 관련 이슈

closed #151

✨ 변경 사항 및 이유

해당 채널에 슬랙 알람이 가도록 수정하였습니다.

✨ PR Point

  • dev.yaml 파일에 slack.url 포함되어있는것을 확인해야합니다.
  • 회원가입 후 슬랙알람이 요청되는지 봐야합니다.

예외상황

  • 회원가입과 추가되어야할 다른 슬랙 알람이 분리되게 하려면 슬랙 채널 url이 추가되어야합니다.

Copy link
Collaborator

@wlwpfh wlwpfh left a comment

Choose a reason for hiding this comment

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

짱입니다 !! 하드 코딩된 값들은 제가 나중에 바꾸겠습니다 ~ 감사합니다 ~

@wlwpfh wlwpfh merged commit e3f1de3 into develop Dec 19, 2023
1 check passed
@wlwpfh wlwpfh deleted the feature/slack-alarm branch February 6, 2024 09:50
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.

[FEAT] 슬랙봇 연결하기
2 participants