fix: add legacy id test to reviewer role check

Author: rakibansary

config/default.js

@@ -70,6 +70,7 @@ module.exports = {
 
   LEGACY_REVIEWER_ROLE_ID: process.env.LEGACY_REVIEWER_ROLE_ID || 4,
   LEGACY_REVIEW_PHASE_ID: process.env.LEGACY_REVIEW_PHASE_ID || 4,
+  LEGACY_REVIEWER_ITERATIVE_ROLE_ID: process.env.LEGACY_REVIEWER_ITERATIVE_ROLE_ID || 21,
   LEGACY_REVIEWER_PAYMENT_TYPE_ID: process.env.LEGACY_PROJECT_REVIEW_PAYMENT_TYPE_ID || 3,
   COPILOT_PAYMENT_TYPE: process.env.COPILOT_PAYMENT_TYPE || 'copilot'
 }

src/common/helper.js

@@ -198,7 +198,7 @@ async function forceV4ESFeeder (legacyId) {
 }
 
 function isReviewerRole (roleId) {
-  return roleId === config.REVIEWER_ROLE_ID || roleId === config.ITERATIVE_REVIEWER_ROLE_ID
+  return roleId === config.REVIEWER_ROLE_ID || roleId === config.ITERATIVE_REVIEWER_ROLE_ID || roleId == config.LEGACY_REVIEWER_ROLE_ID || roleId == config.LEGACY_REVIEWER_ITERATIVE_ROLE_ID
 }
 
 module.exports = {