Project

General

Profile

Actions

Task #3399

closed

Notification before recertification ends

Added by Luboš Čábelka over 1 year ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Luboš Čábelka
Target version:
Start date:
06/20/2023
Due date:
% Done:

100%

Estimated time:
Owner:

Description

Goal of this ticket is to change notification before certification ends.
  • Notification can be send to manager of notification approver, or helpdesk - recipients can be changed using script.
  • One recipient can get only 1 notification with information about all recertification tasks for him.
Actions #3

Updated by Luboš Čábelka over 1 year ago

  • Assignee changed from Luboš Čábelka to Ondřej Kopr
  • % Done changed from 70 to 90

Final version is in branch https://git.bcvsolutions.eu/modules/recertification/-/commits/lcabelka/3399-merge-branch
With merged force recertifications.
@kopro can you review it please?

Actions #4

Updated by Ondřej Kopr over 1 year ago

  • Assignee changed from Ondřej Kopr to Luboš Čábelka
  • % Done changed from 90 to 70
I made code review. Code looks good! I just found some minor things that is just nice to have or not? :D :
  • class RecertificationScriptUtils doesnt include author and description,
  • recertificationInfos in notification recertificationDueDateWarning isnt used in product notification (maybe for example of use - or include example into documentation),
  • info isn't probably countable noun - plural infos doesn't exist,
  • tests missing.

Thanks for this nice feature. Included script is nice idea!

Actions #5

Updated by Luboš Čábelka over 1 year ago

  • % Done changed from 70 to 90
  • Description added into class
  • recertificationInfos removed completely (forgotten from implementation).
  • Added junit tests for new parameters and util class.

All in commit: https://git.bcvsolutions.eu/modules/recertification/-/commit/a7d01c7bde583b86ec12d25b195ce48c09143a9d

Actions #6

Updated by Ondřej Kopr over 1 year ago

  • Status changed from In Progress to Resolved
  • % Done changed from 90 to 100

OK

Actions #7

Updated by Peter Štrunc about 1 year ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF