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] 특정 청첩장에 참석의사 등록 API 구현 #40

Merged
merged 4 commits into from
Jan 24, 2025
Merged

[feat] 특정 청첩장에 참석의사 등록 API 구현 #40

merged 4 commits into from
Jan 24, 2025

Conversation

KyunghwanChoi
Copy link
Contributor

@KyunghwanChoi KyunghwanChoi commented Jan 23, 2025 •
edited
Loading

개요

특정 청첩장에 참석의사 등록 API 구현했습니다.
decision, comment 디렉토리명 단수로 변경했으며,
Decision엔티티의 빌더 메서드와 생성자에서 누락된 부분 수정했습니다.

PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다.
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

📣 To Reviewers

image image

@KyunghwanChoi KyunghwanChoi added feat New feature or request 환이 labels Jan 23, 2025
@KyunghwanChoi KyunghwanChoi self-assigned this Jan 23, 2025
@KyunghwanChoi KyunghwanChoi linked an issue Jan 23, 2025 that may be closed by this pull request
[feat] 특정 청첩장에 참석의사 등록 API 구현 #38
Closed
2 tasks
dyk-im
dyk-im approved these changes Jan 23, 2025
View reviewed changes
Copy link
Contributor

@dyk-im dyk-im left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! 코드 모두 확인했고, 문제가 없는 것으로 보입니다! 추후 반환되는 값이 정말 필요한지, 어떻게 에러핸들리을 할지 등 고민해보시면 좋겠습니다!

@dyk-im dyk-im changed the title [feat] #38 특정 청첩장에 참석의사 등록 API 구현 [feat] 특정 청첩장에 참석의사 등록 API 구현 Jan 23, 2025
dogsub
dogsub approved these changes Jan 24, 2025
View reviewed changes
Copy link
Contributor

@dogsub dogsub left a comment

Choose a reason for hiding this comment

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

늦은 시간까지 고생많으셨습니다!

dyk-im
dyk-im requested changes Jan 24, 2025
View reviewed changes
Copy link
Contributor

@dyk-im dyk-im left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! 수정할 부분이 조금 있는 것 같아 리뷰남겼고 저희가 삭제 기능이 없어 작성해주신 함수는 일단은 지우는 것이 맞다고 판단했습니다!

dyk-im
dyk-im approved these changes Jan 24, 2025
View reviewed changes
Copy link
Contributor

@dyk-im dyk-im left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! 수정사항 모두 확인했고 테스트 결과 문제없이 잘 작동되는 것으로 확인되었습니다!

@dyk-im dyk-im merged commit dddd0af into develop Jan 24, 2025
1 check failed
@dyk-im dyk-im deleted the feat/#38-feat-특정-청첩장에-참석의사-등록-api-구현 branch January 24, 2025 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dyk-im dyk-im dyk-im approved these changes

@dogsub dogsub dogsub approved these changes

@leeseulgi0208 leeseulgi0208 Awaiting requested review from leeseulgi0208

Assignees

@KyunghwanChoi KyunghwanChoi

Labels
feat New feature or request 환이
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[feat] 특정 청첩장에 참석의사 등록 API 구현
3 participants
@KyunghwanChoi @dyk-im @dogsub