Merge pull request #84 from zippick/develop

[DEPLOY] 재배포

Author: price126

zippick/src/main/java/zippick/domain/notification/controller/NotificationController.java

@@ -32,15 +32,14 @@ public ResponseEntity<SendNotificationResponse> sendNotification(
 
     @GetMapping
     public ResponseEntity<ReadNotificationResponse> getNotifications(
-            HttpServletRequest request,
-            @RequestParam int offset) {
+            HttpServletRequest request) {
 
         Long memberId = (Long) request.getAttribute("memberId");
         if (memberId == null) {
             throw new IllegalStateException("memberId가 누락되었습니다.");
         }
 
-        ReadNotificationResponse response = notificationService.getNotifications(memberId, offset);
+        ReadNotificationResponse response = notificationService.getNotifications(memberId);
         return ResponseEntity.ok(response);
     }
 }

zippick/src/main/java/zippick/domain/notification/mapper/NotificationMapper.java

@@ -9,7 +9,6 @@
 public interface NotificationMapper {
     void insertNotification(NotificationDto notification);
 
-    List<NotificationDto> findNotificationsByMemberId(@Param("memberId") Long memberId,
-            @Param("offset") int offset);
+    List<NotificationDto> findNotificationsByMemberId(@Param("memberId") Long memberId);
 
 }

zippick/src/main/java/zippick/domain/notification/service/NotificationService.java

@@ -7,7 +7,7 @@
 public interface NotificationService {
     SendNotificationResponse sendNotification(Long memberId, NotificationRequest request);
 
-    ReadNotificationResponse getNotifications(Long memberId, int offset);
+    ReadNotificationResponse getNotifications(Long memberId);
 
 }
 

zippick/src/main/java/zippick/domain/notification/service/NotificationServiceImpl.java

@@ -42,8 +42,8 @@ public SendNotificationResponse sendNotification(Long memberId, NotificationRequ
 
     @Override
     @Transactional(readOnly = true)
-    public ReadNotificationResponse getNotifications(Long memberId, int offset) {
-        List<NotificationDto> list = notificationMapper.findNotificationsByMemberId(memberId, offset);
+    public ReadNotificationResponse getNotifications(Long memberId) {
+        List<NotificationDto> list = notificationMapper.findNotificationsByMemberId(memberId);
         return new ReadNotificationResponse(list);
     }