adamsaghy commented on code in PR #2330:
URL: https://github.com/apache/fineract/pull/2330#discussion_r877005486
##########
fineract-provider/src/main/java/org/apache/fineract/notification/service/NotificationWritePlatformServiceImpl.java:
##########
@@ -64,7 +53,7 @@ public Long notify(Long userId, String objectType, Long
objectIdentifier, String
private Long insertIntoNotificationMapper(Long userId, Long
generatedNotificationId) {
AppUser appUser = this.appUserRepository.findById(userId).orElse(null);
NotificationMapper notificationMapper = new NotificationMapper(
-
this.notificationGeneratorReadRepositoryWrapper.findById(generatedNotificationId),
appUser, false, getCurrentDateTime());
+
this.notificationGeneratorReadRepositoryWrapper.findById(generatedNotificationId),
appUser, false, new Date());
Review Comment:
Changed my mind. you can leave it as is for now.. there are already a bunch
of places where the created_date system date and some places where it is tenant
date. We shall figure those out later in a generic, common way
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
