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.

Sign up for GitHub

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

Jump to bottom

fix : 호스트 이벤트 버그 수정 및 슬랙 URL 검증 추가 #348

Merged
merged 6 commits into from
Feb 12, 2023
Merged

fix : 호스트 이벤트 버그 수정 및 슬랙 URL 검증 추가 #348

merged 6 commits into from
Feb 12, 2023

Conversation

gengminy
Copy link
Member

@gengminy gengminy commented Feb 11, 2023 •
edited
Loading

개요

작업사항

  • 이벤트 준비중일 때 열람 권한을 속한 유저 -> 초대 수락한 유저로 변경
  • HostInfoVo 에서 누락된 partner 필드 추가
  • 슬랙 메세지 이벤트 핸들러 디렉토리 이동
  • SlackURL 필드 NULL CHECK
  • 호스트의 슬랙 URL 등록 시 ack 가 올바르게 오지 않을 경우 등록 취소시키는 기능 추가

image

  • 호스트 슬랙 url 과 변경하려는 url 의 중복 검사를 먼저 하기 위해 validation 로직을 usecase 로 올렸습니다

image

  • 슬랙 url 등록과 메세지 전송을 분리시켰습니다
  • 슬랙 url 이 존재했다가 없어질 경우 로그에 에러 메세지가 계속 뜰텐데, 이를 생각해서 일단 임시로 메세지 전송에서는 임시로 예외를 무시하도록 했습니다.
  • url 등록은 1회성이니 동기로 처리해서 메세지 성공 시 받을 응답을 대기하도록 했습니다.

변경로직

  • 위와 같음

@gengminy gengminy added For: API [이슈 대상] 외부 API Type: Errors/Bugs [이슈 목적] 버그, 애러 수정 Type: Feature [이슈 목적] 새로운 기능 추가 labels Feb 11, 2023
@gengminy gengminy added this to the 두둥 스프린트 2차 milestone Feb 11, 2023
@gengminy gengminy self-assigned this Feb 11, 2023
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 315 Code Smells

15.9% 15.9% Coverage
0.0% 0.0% Duplication

cofls6581
cofls6581 approved these changes Feb 11, 2023
View reviewed changes
Copy link
Member

@cofls6581 cofls6581 left a comment

Choose a reason for hiding this comment

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

LGTM!

ImNM
ImNM approved these changes Feb 12, 2023
View reviewed changes
Copy link
Member

@ImNM ImNM left a comment

Choose a reason for hiding this comment

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

LGTM

@gengminy gengminy merged commit ccd04d4 into dev Feb 12, 2023
@gengminy gengminy deleted the bug/337-event branch February 15, 2023 04:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cofls6581 cofls6581 cofls6581 approved these changes

@ImNM ImNM ImNM approved these changes

@sanbonai06 sanbonai06 Awaiting requested review from sanbonai06 sanbonai06 is a code owner

@kim-wonjin kim-wonjin Awaiting requested review from kim-wonjin kim-wonjin is a code owner

Assignees

@gengminy gengminy

Labels
For: API [이슈 대상] 외부 API Type: Errors/Bugs [이슈 목적] 버그, 애러 수정 Type: Feature [이슈 목적] 새로운 기능 추가
Projects
두둥
Status: Done
Milestone
두둥 스프린트 2차
Development

Successfully merging this pull request may close these issues.

🐛[BUG] 슬랙 URL NULL 체크 오류
3 participants
@gengminy @ImNM @cofls6581