An error occurred while fetching the assigned milestone of the selected merge_request.
refacto(NotificationType): [AOU-543] simplify notification types and relation to subscribers
1 unresolved thread
https://issues.unige.ch/browse/AOU-543
Can be tested with a fresh database or use SQL scripts on https://doc.unige.ch/display/AO/%5BAoU%5D+Environment+upgrade
Edited by Nicolas.Rod
Merge request reports
Activity
added 4 commits
-
7389c478...8c03c28f - 3 commits from branch
master
- 4e66b963 - refacto(NotificationType): [AOU-543] simplify notification types and relation to subscribers
-
7389c478...8c03c28f - 3 commits from branch
added 3 commits
-
4e66b963...20bcf9e1 - 2 commits from branch
master
- e3b9ca6d - refacto(NotificationType): [AOU-543] simplify notification types and relation to subscribers
-
4e66b963...20bcf9e1 - 2 commits from branch
added 1 commit
- d6f9607b - fix(NotificationDoc): set recipient resId to allow building
added 4 commits
-
d6f9607b...a3956e8a - 2 commits from branch
master
- 01ce0594 - refacto(NotificationType): [AOU-543] simplify notification types and relation to subscribers
- cb624483 - fix(NotificationDoc): set recipient resId to allow building
-
d6f9607b...a3956e8a - 2 commits from branch
71 Notification createRemoteNotification(Notification localNotification) { 72 return this.notificationService.create(localNotification); 38 return publication; 73 39 } 74 40 75 void inboxTest(NotificationType notificationType, boolean shouldBeInInbox, boolean isInResearchGroup) { 76 final Notification localNotification = this.createLocalNotification(notificationType, isInResearchGroup); 77 final Notification remoteNotification = this.createRemoteNotification(localNotification); 78 final Notification inboxNotification = this.notificationService.getInboxNotification(remoteNotification.getResId()); 79 assertEquals(inboxNotification != null, shouldBeInInbox, "Notification created should be in inbox"); 80 if (shouldBeInInbox) { 81 final List<Notification> notificationList = this.notificationService.getInboxNotifications(); 82 assertTrue(notificationList.size() >= 1, "There should be at least one notification in the inbox"); 83 assertTrue(notificationList.stream().anyMatch(n -> n.getResId().equals(inboxNotification.getResId())), 84 "There should be a least one notification with id= "+ inboxNotification.getResId()); 41 protected Notification getLastNotificationForCurrentUser() { - Resolved by Nicolas.Rod
added 6 commits
-
cb624483...9ac52d46 - 4 commits from branch
master
- 690dc2bf - refacto(NotificationType): [AOU-543] simplify notification types and relation to subscribers
- 27eee600 - fix(NotificationDoc): set recipient resId to allow building
-
cb624483...9ac52d46 - 4 commits from branch
added 1 commit
- 731a048d - fix(NotificationService): remove not used repository
Please register or sign in to reply