Skip to content

Commit

Permalink
feat: 최근 알림 목록 조회 로직 수정(#66)
Browse files Browse the repository at this point in the history
- 페이지 번호가 optional해짐에 따라 요청 DTO 검증 로직 추가
- 읽음 처리 메서드명 변경에 따른 사용 위치 수정
  • Loading branch information
Minjae-An committed May 11, 2024
1 parent 4c792b7 commit 4b428f7
Showing 1 changed file with 6 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@

import java.util.Arrays;
import java.util.List;
import java.util.Objects;
import java.util.Optional;
import lombok.RequiredArgsConstructor;
import org.springframework.data.domain.Page;
Expand Down Expand Up @@ -40,7 +41,10 @@ public NotificationPageResponse getRecentlyReceivedNotificationsBy(
long receiverId, NotificationPageRequest request) {

// 페이지 번호는 optional, default 첫 페이지 제공
int pageNumber = Optional.ofNullable(request.getPage()).orElse(0);
int pageNumber = 0;
if (Objects.nonNull(request) && Objects.nonNull(request.getPage())) {
pageNumber = request.getPage();
}
int size = 45;

PageRequest pageRequest = PageRequest.of(pageNumber, size);
Expand All @@ -56,7 +60,7 @@ public NotificationPageResponse getRecentlyReceivedNotificationsBy(
.orElse(Long.MAX_VALUE);

// 가장 최근에 받은 알림 포함 이전 알림 전부 읽음 처리
notificationCommand.readAllHasIdLessThanEqual(mostRecentNotificationId);
notificationCommand.readNotificationsUpToIdBy(receiverId, mostRecentNotificationId);

return NotificationPageResponse.from(page);
}
Expand Down

0 comments on commit 4b428f7

Please sign in to comment.