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: 초대장 생성/상세조회 요청, 응답 DTO 수정 및 초대장 엔티티 추가 #40

Merged
merged 9 commits into from
Feb 3, 2025

Conversation

1winhyun
Copy link
Collaborator

@1winhyun 1winhyun commented Feb 2, 2025

✅ PR 유형

어떤 변경 사항이 있었나요?

  • 새로운 기능 추가
  • 버그 수정
  • 리팩토링
  • 코드에 영향을 주지 않는 변경사항(주석, 개행 등등..)
  • 문서 수정
  • 빌드 부분 혹은 패키지 매니저 수정
  • 테스트 코드 추가

✏️ 작업 내용

**클라이언트 측의 요청에 따라 초대장 생성 기능에서 응답 DTO의 경우 invitationID만 반환되도록 수정하였습니다.
image

**InvitationEntity에 templateKey Column을 추가하였으며 null값을 허용하였습니다. **
image

다음과 같이 값을 넣어도, 아무것도 넣지 않아도(null) 잘 동작하게 구현하였습니다.
image
image

다음과 같이 초대장 상세조회에도 templateKey가 나타나도록 구현하였습니다.
image


🔗 관련 이슈


💡 추가 사항

클라이언트 측 추가 요구사항으로 초대장 상세조회 Response 값에 초대장 생성자의 nickname을 추가하였습니다.
image

@1winhyun 1winhyun self-assigned this Feb 2, 2025
Copy link

github-actions bot commented Feb 2, 2025

Test Results

31 files  31 suites   2s ⏱️
64 tests 64 ✅ 0 💤 0 ❌
66 runs  66 ✅ 0 💤 0 ❌

Results for commit 7c975a2.

♻️ This comment has been updated with latest results.

@1winhyun 1winhyun requested review from yechan-kim and jwnnoh February 2, 2025 16:22
@1winhyun 1winhyun added ♻️ refactor 코드 리팩토링 ✅ test 테스트 코드 labels Feb 2, 2025
Copy link
Member

@yechan-kim yechan-kim left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!
승현님 덕분에, 코드의 간결성이 높아졌네요!!!
간단한 코멘트 남겨두었으니, 읽어보시고 답변 부탁드립니다!

Copy link
Member

@yechan-kim yechan-kim left a comment

Choose a reason for hiding this comment

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

의견 남겨두었습니다!
확인 부탁드립니다!!!

Copy link
Member

@jwnnoh jwnnoh left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! 의견 남겼으니 봐주시면 감사하겠습니다..!

Copy link
Member

@jwnnoh jwnnoh left a comment

Choose a reason for hiding this comment

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

👍

@1winhyun 1winhyun merged commit 1581657 into main Feb 3, 2025
2 checks passed
@jwnnoh jwnnoh deleted the refactor/#39 branch February 3, 2025 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ refactor 코드 리팩토링 ✅ test 테스트 코드
Projects
None yet
Development

Successfully merging this pull request may close these issues.

✨ 공통 응답 객체 구현
3 participants