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

feat : PromiseRegisterEventHandler 구현 #88

Merged
merged 1 commit into from
Jun 14, 2023
Merged

feat : PromiseRegisterEventHandler 구현 #88

merged 1 commit into from
Jun 14, 2023

Conversation

BlackBean99
Copy link
Collaborator

개요

작업사항

  • 약속 생성시 방장의 PromiseUser default 생성하게 했습니다.

변경로직

  • PromiseRegisterEvents에 userId가 필요해서 추가했습니다.
    다들 참고해주세요
class PromiseRegisterEvent(
    val promiseId: Long,
    val userId: Long,
) : DomainEvent()

@BlackBean99 BlackBean99 added function 특정 기능 추가 enhancement New feature or request labels Jun 14, 2023
@sonarcloud
Copy link

sonarcloud bot commented Jun 14, 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 0 Code Smells

30.0% 30.0% Coverage
0.0% 0.0% Duplication

@BlackBean99 BlackBean99 merged commit 6bb55d5 into develop Jun 14, 2023
@BlackBean99 BlackBean99 deleted the feature/DPMBE-59 branch June 14, 2023 09:27
kdomo pushed a commit that referenced this pull request Jun 16, 2023
@BlackBean99 @kdomo
feat : PromiseRegisterEventHandler 구현 (#88)
f256e94
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ImNM ImNM ImNM approved these changes

@kdomo kdomo kdomo approved these changes

Assignees

@BlackBean99 BlackBean99

Labels
enhancement New feature or request function 특정 기능 추가
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[DPMBE-59] 첫 약속 생성시 PromiseUser 자동 생성 EventHandling
3 participants
@BlackBean99 @ImNM @kdomo