fix(recommendationsEmails): EQA emails logic

packages/component-manuscript-manager/src/routes/fragmentsRecommendations/post.js

@@ -145,16 +145,12 @@ module.exports = models => async (req, res) => {
     const hasPeerReview = !isEmpty(collection.handlingEditor)
 
     if (isEditorInChief) {
-      if (
-        recommendation === 'publish' &&
-        collection.technicalChecks.token &&
-        ['accepted', 'inQA'].includes(collection.status)
-      ) {
-        if (collection.status === 'inQA') {
-          notification.notifyEAWhenEiCRequestsEQAApproval()
-        } else {
-          notification.notifyEAWhenEiCMakesFinalDecision()
-        }
+      if (recommendation === 'publish' && collection.status === 'inQA') {
+        notification.notifyEAWhenEiCRequestsEQAApproval()
+      }
+
+      if (recommendation === 'publish' && collection.status === 'accepted') {
+        notification.notifyEAWhenEiCMakesFinalDecision()
       }
 
       if (hasPeerReview && (recommendation !== 'publish' || hasEQA)) {

packages/component-manuscript-manager/src/routes/technicalChecks/patch.js

@@ -51,7 +51,6 @@ module.exports = ({ Collection, Fragment, User }) => async (req, res) => {
       })
     }
 
-    delete collection.technicalChecks.token
     if (agree) {
       if (step === TECHNICAL_STEPS.EQA) {
         collection.technicalChecks.eqa = true
@@ -61,6 +60,7 @@ module.exports = ({ Collection, Fragment, User }) => async (req, res) => {
       }
     }
 
+    delete collection.technicalChecks.token
     collection.status = setNewStatus(step, agree)
     await collection.save()