Skip to content

Task email notifications - email bugs fixes

Vaibhav Rathore requested to merge feature/1196/email-bug-fix into main

Addresses the following issues (not tracked on GitHub) Screenshot_2023-03-02_at_14.05.13

  1. A user may not have any role attached against a manuscript. Hence the isEditor/isReviewer check will check for a non-null check
  2. There was an issue with the persistedTasks array (was happening in local) where it was giving an error as the array was readonly. Hence, added a slice() operator to make a copy of it. Fixed by this StackOverflow answer
  3. Dropped the unique index set for recipient and email notification task id which was restricting a task to only have one entry for a recipient in the list of emails.
Edited by Vaibhav Rathore

Merge request reports