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

[BE] 약속 생성 API에서 바디를 통해 uuid를 제공 #87

Merged
merged 2 commits into from
Jul 25, 2024

Conversation

ehBeak
Copy link
Contributor

@ehBeak ehBeak commented Jul 25, 2024

관련 이슈

작업 내용

약속을 생성할 때, 약속의 uuid를 응답 바디에 넣어 보냅니다.

특이 사항

리뷰 요구사항 (선택)

Co-authored-by: seunghye218 <seunghye@student.42seoul.kr>
@ehBeak ehBeak self-assigned this Jul 25, 2024
@ehBeak ehBeak added 🐈‍⬛ 백엔드 백엔드 관련 이슈에요 :) 🥵 핫픽스 핫! 빠르게 문제를 해결해요 :( ♻️ 리팩터링 코드를 깎아요 :) and removed 🥵 핫픽스 핫! 빠르게 문제를 해결해요 :( labels Jul 25, 2024
Copy link

github-actions bot commented Jul 25, 2024

Test Results

43 tests   43 ✅  4s ⏱️
14 suites   0 💤
14 files     0 ❌

Results for commit 1e6bff0.

♻️ This comment has been updated with latest results.

public ResponseEntity<MomoApiResponse<MeetingSharingResponse>> create(
@RequestBody @Valid MeetingCreateRequest request
) {
MeetingSharingResponse response = meetingService.create(request);
Copy link
Member

Choose a reason for hiding this comment

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

현재 임시 구현일수도 있지만, 지금과 같은 맥락에서 body에 UUID를 담아 응답하는 구현이 계속 유지된다면 나중에 MeetingSharingResponse라는 레코드명은 수정이 필요할 것 같아요!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

그렇네요! 이후 분리하도록 하겠습니다~

@ehBeak ehBeak added this to the 2차 데모데이 milestone Jul 25, 2024
@ehBeak ehBeak merged commit 5d9433c into develop Jul 25, 2024
6 checks passed
@ehBeak ehBeak deleted the hotfix/83-fix-response branch July 27, 2024 04:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ 리팩터링 코드를 깎아요 :) 🐈‍⬛ 백엔드 백엔드 관련 이슈에요 :)
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[BE] 약속 생성 API에서 바디를 통해 uuid를 제공해요 :)
5 participants