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

refactor : 호스트 검증 aop 적용 #288

Merged
merged 1 commit into from
Feb 3, 2023
Merged

Conversation

gengminy
Copy link
Member

@gengminy gengminy commented Feb 3, 2023

개요

작업사항

  • 호스트에 검증 AOP 적용 및 리팩토링
  • 이벤트에 검증 AOP 적용 및 리팩토링

변경로직

  • 위와 같음

@gengminy gengminy added For: API [이슈 대상] 외부 API Type: Refactor [이슈 목적] 프로덕션 코드 리팩토링 labels Feb 3, 2023
@gengminy gengminy requested a review from ImNM as a code owner February 3, 2023 07:06
@gengminy gengminy self-assigned this Feb 3, 2023
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.

키야~ 소스가 아주 이뻐지는 구만유~

@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 3, 2023

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 292 Code Smells

15.6% 15.6% Coverage
0.0% 0.0% Duplication

Copy link
Member

@sanbonai06 sanbonai06 left a comment

Choose a reason for hiding this comment

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

이번 이슈에서 저도 달아보겠습니다!

@gengminy gengminy merged commit 6528331 into dev Feb 3, 2023
@gengminy gengminy deleted the refactor/260-host-aop branch February 3, 2023 07:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
For: API [이슈 대상] 외부 API Type: Refactor [이슈 목적] 프로덕션 코드 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🔨[refactor] 호스트 검증 AOP 적용
3 participants