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

[feat] 약속 멤버 엔티티 구현 #32

Merged
merged 2 commits into from
Feb 13, 2025
Merged

[feat] 약속 멤버 엔티티 구현 #32

merged 2 commits into from
Feb 13, 2025

Conversation

0-tae
Copy link
Contributor

@0-tae 0-tae commented Feb 12, 2025

🚀 What’s this PR about?

  • 작업 내용 요약: 약속 멤버 엔티티를 구현 하였습니다.
  • 핵심 변경사항: 약속 멤버 엔티티 및 VO 구현 하였습니다.

🛠️ What’s been done?

  • 주요 변경사항을 상세히 기술하세요. (예: 새로운 기능 추가, 버그 수정, 코드 리팩토링 등)
    • 약속 멤버 엔티티 구현
    • VO 구현

🧪 Testing Details

  • 테스트는 별도로 진행하지 않았습니다.

👀 Checkpoints for Reviewers

  • 리뷰 시 확인할 사항:
    • AvailabilityStatus의 값을 임의로 지정했습니다. 혹시 따로 생각해 두신 부분이 있을까요?

🎯 Related Issues

@0-tae 0-tae requested a review from jsoonworld February 12, 2025 02:13
@0-tae 0-tae self-assigned this Feb 12, 2025
@jsoonworld jsoonworld added ✨ feat Something isn't working 🦖 용태 🦖 Further information is requested labels Feb 12, 2025
Comment on lines 6 to 14
@Getter
@AllArgsConstructor
public enum AvailabilityStatus {

ABLE("참여"),
NOT_ABLE("불참");

private final String message;
}
Copy link
Member

Choose a reason for hiding this comment

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

AppointmentAvailability로 VO 이름을 변경하는 건 어떨까요?

이렇게 하면

AVAILABLE("약속 가능"),
UNAVAILABLE("약속 불가능");

이런 형태로 변경할 수 있을 것 같네요.

또, availability 자체가 상태를 나타내기 때문에 status가 불필요할 수도 있다는 생각이 들었어요.
이 enum은 약속 멤버가 해당 약속에 대해 가능한지, 불가능한지를 나타내는 역할이라 AppointmentAvailability라는 이름도 적절해 보이는데, 어떻게 생각하시나요? 😊

Copy link
Contributor Author

Choose a reason for hiding this comment

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

좋은 생각입니다. 그럼 제안 하신대로 변경하고, merge 하겠습니다 😀

@0-tae 0-tae merged commit 606717b into develop Feb 13, 2025
1 check passed
@0-tae 0-tae deleted the feat/24 branch February 13, 2025 00:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feat Something isn't working 🦖 용태 🦖 Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants