Task #1933
closedCreate LRT for remove old notifications.
100%
Description
Create LRT for remove old notifications (all types) and related recipients.
Files
Related issues
Updated by Radek Tomiška almost 5 years ago
- Status changed from In Progress to Needs feedback
- Assignee changed from Radek Tomiška to Vít Švanda
- % Done changed from 0 to 90
Notifications can be removed and long running task for remove sent notifications (''DeleteNotificationTaskExecutor'') older then 180 days is scheduled by default.
Commit:
https://github.com/bcvsolutions/CzechIdMng/commit/5c90ce9fdf00ab1afe3d841adb4bff9a01cb7e0e
Doc:
https://wiki.czechidm.com/devel/documentation/application_configuration/dev/scheduled_tasks/task-scheduler#deletenotificationtaskexecutor
https://github.com/bcvsolutions/CzechIdMng/blob/hotfix-9.7.12/CHANGELOG.md#9712
Could you do a feedback, please?
Updated by Vít Švanda almost 5 years ago
- File notifi-delete.png notifi-delete.png added
- Status changed from Needs feedback to In Progress
- Assignee changed from Vít Švanda to Radek Tomiška
I did review and test. Works correctly.
I disovered only one strange behavior in count of deleted notifications. I deleted all notifications older then 360 days. Initial count in LRT was 270 but on the end were deleted only 244 notifications (without any exceptions). I do not see reason why is count different in this cause.
Second executing of the same LRT didn't delete any notifications.
Updated by Radek Tomiška almost 5 years ago
The reason is, if the parent notification is deleted before children and children is deleted too with the parent (cascade). Then is counter not incremented (childen is not found in the next page). I'll try to add sort by parent to ensure child is deleted at first.
Updated by Radek Tomiška almost 5 years ago
- Status changed from In Progress to Needs feedback
- Assignee changed from Radek Tomiška to Vít Švanda
Child notifications are removed at first, now even count should fit, commit:
https://github.com/bcvsolutions/CzechIdMng/commit/54907e3a8d0d40730fa7bed07f3cd6093828b4b2
Could you do a feedback please, again :)
Updated by Vít Švanda almost 5 years ago
- Status changed from Needs feedback to Resolved
- Assignee changed from Vít Švanda to Radek Tomiška
- % Done changed from 90 to 100
I tested it. Start and end count match now, thanks for that.
Updated by Radek Tomiška almost 5 years ago
- Status changed from Resolved to Closed
Updated by Radek Tomiška over 4 years ago
- Related to Feature #1671: Remove old notification added