New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Pending assessments email notification #3231

Closed

VassilIordanov opened this issue Oct 5, 2020 · 2 comments · Fixed by #3286

Assignees

Labels

Contributor

VassilIordanov commented Oct 5, 2020 •

edited by gjvoosten

Loading

The users that have new pending assessments should receive:

an email as soon as the assessments are available to be made
a reminder email:
- for daily recurrence, never
- for weekly recurrence, 3 days after notification
- for biweekly recurrence, 5 days after notification
- for semimonthly recurrence, 5 days after notification
- for monthly recurrence, 1 week after notification
- for quarterly recurrence, 4 weeks after notification
- for semiannually recurrence, 1 month after notification
- for annually recurrence, 1 month after notification

VassilIordanov added the enhancement label

VassilIordanov self-assigned this

Contributor

cemalettin-work commented Oct 6, 2020

This is related to the issue #3208 and to the PR #3212 . We should think about this when solving previous ones.

cemalettin-work linked a pull request

that will close this issue

Add notifications for pending assessments #3212

Merged

14 tasks

VassilIordanov closed this as completed in #3212

Contributor

cemalettin-work commented Oct 19, 2020

This shouldn't have been closed automatically

cemalettin-work reopened this

cemalettin-work removed a link to a pull request

Add notifications for pending assessments #3212

Merged

14 tasks

cemalettin-work self-assigned this

gjvoosten self-assigned this

cemalettin-work removed their assignment

gjvoosten added a commit that referenced this issue


          #3231: Pull up logger, and always pass AnetConfiguration to constructor

Move worker creation closer to where it's scheduled.
Make some variables final.

gjvoosten added a commit that referenced this issue


          #3231: Add missing equals and hashCode

617715b

gjvoosten added a commit that referenced this issue


          #3231: Prevent NPE when getting a non-existing key path

2fb1e3c

gjvoosten added a commit that referenced this issue


          #3231: Enhance method to get either all active top- or sub-level tasks

7942d75

gjvoosten added a commit that referenced this issue


          #3231: Add method to safely parse JSON

69ee9a2

gjvoosten added a commit that referenced this issue


          #3231: Move metricRegistry to AnetObjectEngine

85d364c

gjvoosten added a commit that referenced this issue


          #3231: Introduce a batching & caching context for workers

c7f6f04

gjvoosten added a commit that referenced this issue


          #3231: Make relevant parts of some workers asynchronous

44d476c

This gives the context the opportunity to batch queries.

gjvoosten added a commit that referenced this issue


          #3231: WIP: Add PendingAssessmentsNotificationWorker

9c9a112

gjvoosten added a commit that referenced this issue


          #3231: WIP: Add PendingAssessmentsNotificationWorker

df2be33

gjvoosten added a commit that referenced this issue


          #3231: WIP: Add PendingAssessmentsNotificationWorker

4bf25a3

gjvoosten added a commit that referenced this issue


          #3231: WIP: Add PendingAssessmentsNotificationWorker

a7ade26

gjvoosten added a commit that referenced this issue


          #3231: WIP: Add PendingAssessmentsNotificationWorker

eb81052

gjvoosten added a commit that referenced this issue


          #3231: Pull up logger, and always pass AnetConfiguration to constructor

50d35ef

Move worker creation closer to where it's scheduled.
Make some variables final.

gjvoosten added a commit that referenced this issue


          #3231: Add missing equals and hashCode

27d26ea

gjvoosten added a commit that referenced this issue


          #3231: Prevent NPE when getting a non-existing key path

00ee232

gjvoosten added a commit that referenced this issue


          #3231: Enhance method to get either all active top- or sub-level tasks

819b40c

gjvoosten added a commit that referenced this issue


          #3231: Add method to safely parse JSON

0d7b0fa

gjvoosten added a commit that referenced this issue


          #3231: Move metricRegistry to AnetObjectEngine

69591bf

gjvoosten added a commit that referenced this issue


          #3231: Make relevant parts of some workers asynchronous

a4463a6

This gives the context the opportunity to batch queries.

gjvoosten added a commit that referenced this issue


          #3231: Add PendingAssessmentsNotificationWorker

a70f430

gjvoosten added a commit that referenced this issue


          #3231: Pull up logger, and always pass AnetConfiguration to constructor

262def6

Move worker creation closer to where it's scheduled.
Make some variables final.

gjvoosten added a commit that referenced this issue


          #3231: Add missing equals and hashCode

881b347

gjvoosten added a commit that referenced this issue


          #3231: Prevent NPE when getting a non-existing key path

f9b05a4

gjvoosten added a commit that referenced this issue


          #3231: Enhance method to get either all active top- or sub-level tasks

ee0ae98

gjvoosten added a commit that referenced this issue


          #3231: Add method to safely parse JSON

256923a

gjvoosten added a commit that referenced this issue


          #3231: Move metricRegistry to AnetObjectEngine

c587c7a

gjvoosten added a commit that referenced this issue


          #3231: Introduce a batching & caching context for workers

5f30f5c

gjvoosten added a commit that referenced this issue


          #3231: Make relevant parts of some workers asynchronous

d8b9f0d

This gives the context the opportunity to batch queries.

gjvoosten added a commit that referenced this issue


          #3231: Add PendingAssessmentsNotificationWorker

f944fb5

gjvoosten added a commit that referenced this issue


          #3231: Pull up logger, and always pass AnetConfiguration to constructor

bbb3efb

Move worker creation closer to where it's scheduled.
Make some variables final.

gjvoosten added a commit that referenced this issue


          #3231: Prevent NPE when getting a non-existing key path

00a123a

gjvoosten added a commit that referenced this issue


          #3231: Enhance method to get either all active top- or sub-level tasks

c4838ac

gjvoosten added a commit that referenced this issue


          #3231: Add method to safely parse JSON

fa2800a

gjvoosten added a commit that referenced this issue


          #3231: Move metricRegistry to AnetObjectEngine

bfb6510

gjvoosten added a commit that referenced this issue


          #3231: Introduce a batching & caching context for workers

d79131b

gjvoosten added a commit that referenced this issue


          #3231: Make relevant parts of some workers asynchronous

b764ad2

This gives the context the opportunity to batch queries.

gjvoosten added a commit that referenced this issue


          #3231: Add PendingAssessmentsNotificationWorker

8bb54cf

gjvoosten added a commit that referenced this issue


          #3231: Add PendingAssessmentsNotificationWorker

3fd0c88

gjvoosten added a commit that referenced this issue


          #3231: Pull up logger, and always pass AnetConfiguration to constructor

8c0800f

Move worker creation closer to where it's scheduled.
Make some variables final.

gjvoosten added a commit that referenced this issue


          #3231: Prevent NPE when getting a non-existing key path

05a3eb5

gjvoosten added a commit that referenced this issue


          #3231: Enhance method to get either all active top- or sub-level tasks

e05fc69

gjvoosten added a commit that referenced this issue


          #3231: Add method to safely parse JSON

246ebe6

gjvoosten added a commit that referenced this issue


          #3231: Move metricRegistry to AnetObjectEngine

a105402

gjvoosten added a commit that referenced this issue


          #3231: Introduce a batching & caching context for workers

b3cd57e

gjvoosten added a commit that referenced this issue


          #3231: Make relevant parts of some workers asynchronous

1f98e76

This gives the context the opportunity to batch queries.

gjvoosten added a commit that referenced this issue


          #3231: Add PendingAssessmentsNotificationWorker

49f3983

VassilIordanov closed this as completed in #3286

VassilIordanov added a commit that referenced this issue


          Merge pull request #3286 from NCI-Agency/GH-3231-pending-assessments-…

8b61b70

…email-notification

Send email notifications for pending assessments

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment