Merge pull request #46 from lotteon2/feat/read-notification

fix: 알림 읽음 처리 수정
lotteon2 · Jan 17, 2024 · 4f8bc44 · 4f8bc44
2 parents 142db50 + 7cb2434
commit 4f8bc44
Showing 1 changed file with 5 additions and 6 deletions.
diff --git a/...fication/domain/notification/infrastructure/http/api/NotificationQueryRestController.java b/...fication/domain/notification/infrastructure/http/api/NotificationQueryRestController.java
@@ -72,14 +72,13 @@ public void customerNotificationCheck(
   }
 
   @PutMapping("admin/check")
-  public void adminNotificationCheck(
-      @RequestBody List<Long> notificationId, @RequestHeader Long userId) {
-    notificationCommandService.updateNotificationIsRead(notificationId, userId, Role.ADMIN);
+  public void adminNotificationCheck(@RequestBody List<Long> notificationId) {
+    notificationCommandService.updateNotificationIsRead(notificationId, 100L, Role.ADMIN);
   }
 
-  @PutMapping("manager/check")
+  @PutMapping("manager/{storeId}/check")
   public void managerNotificationCheck(
-      @RequestBody List<Long> notificationId, @RequestHeader Long userId) {
-    notificationCommandService.updateNotificationIsRead(notificationId, userId, Role.MANAGER);
+      @RequestBody List<Long> notificationId, @PathVariable Long storeId) {
+    notificationCommandService.updateNotificationIsRead(notificationId, storeId, Role.MANAGER);
   }
 }