-
Notifications
You must be signed in to change notification settings - Fork 1
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
[DPMBE-127] 알림 하이라이트를 조회한다 #228
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.
수고하셨슴다
@@ -27,4 +31,14 @@ class NotificationController( | |||
): NotificationResponse { | |||
return notificationReadUseCase.execute(pageable) | |||
} | |||
|
|||
@Operation(summary = "하이라이트 조회", description = "약속 ID로 하이라이트를 조회합니다") | |||
@GetMapping("/highlights/promises/{promiseId}") |
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.
parameter 에서 카멜케이스말고 저는 promise-id 이렇게 작성했었습니다 참고!
@GetMapping("/highlights/promises/{promiseId}") | ||
fun getHighlights( | ||
@PathVariable promiseId: Long, | ||
@ParameterObject @PageableDefault |
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.
@PageableDefault(size=10) 이렇게 기본값을 설정해줘도 좋을 것 같아용
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.
명시적으로 적어주는게!! 가독성이 좋지 않을까여? 10이 아니더라도 ㅋㅋ
Kudos, SonarCloud Quality Gate passed! |
개요
작업사항