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

Feature 36/알림 여부 #39

Merged
merged 4 commits into from
Jul 28, 2024
Merged

Feature 36/알림 여부 #39

merged 4 commits into from
Jul 28, 2024

Conversation

Ryeolee
Copy link
Contributor

@Ryeolee Ryeolee commented Jul 26, 2024

  1. 수정사항
  • User 엔티티 알람상태 컬럼 추가
  1. 개발 사항
  • User 알람 상태 업데이트 API
  • User 알람 상태 조회 API

@Ryeolee Ryeolee added the development 개발 label Jul 26, 2024
@Ryeolee Ryeolee requested a review from MinchoGreenT July 26, 2024 18:16
@Ryeolee Ryeolee self-assigned this Jul 26, 2024
@@ -80,6 +83,7 @@ public static User createUser(String role, String userNumber, boolean early){
.role(role)
.userNumber(userNumber)
.early(early)
.notificationStatus(true)
Copy link
Contributor

Choose a reason for hiding this comment

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

커밋 메세지에는 디폴트값이 false라고 써져있는데 코드 내용은 다른 것 같네요

Copy link
Contributor Author

Choose a reason for hiding this comment

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

아 커밋 메시지를 잘못썼네요...디폴트 true입니다!

@MinchoGreenT MinchoGreenT merged commit d0dc148 into dev Jul 28, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants