-
Notifications
You must be signed in to change notification settings - Fork 0
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
커피챗 기능 개발 #49
커피챗 기능 개발 #49
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
굳굳~! 수고하셨습니다~ 저거 id name만 수정 부탁드릴게요!
|
||
@Id | ||
@GeneratedValue(strategy = GenerationType.IDENTITY) | ||
@Column(name = "coffeeChat_id") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
카멜케이스와 스네이크 케이스를 혼용해서 사용했어요!
public void create(CoffeeChat coffeeChat, Long toUserId) { | ||
User toUser = userReader.readUser(toUserId); | ||
coffeeChat.updateToUser(toUser); | ||
coffeeChatCreator.create(coffeeChat); | ||
} | ||
|
||
public void accept(User user, Long coffeeChatId) { | ||
CoffeeChat coffeeChat = coffeeChatReader.read(coffeeChatId); | ||
coffeeChatValidator.shouldBeSameUser(user, coffeeChat.getToUser()); | ||
coffeeChatValidator.shouldBePending(coffeeChat); | ||
coffeeChatUpdater.updateState(coffeeChat, State.ACCEPT); | ||
} | ||
|
||
public void reject(User user, Long coffeeChatId) { | ||
CoffeeChat coffeeChat = coffeeChatReader.read(coffeeChatId); | ||
coffeeChatValidator.shouldBeSameUser(user, coffeeChat.getToUser()); | ||
coffeeChatValidator.shouldBePending(coffeeChat); | ||
coffeeChatUpdater.updateState(coffeeChat, State.REJECT); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
다 깔끔하고 너무 좋은 것 같습니다 🥹
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
굳굳~
🎫 관련 이슈
close #38
📄 개요
🔨 작업 내용
🏁 확인 사항
🙋🏻 덧붙일 말