Merge pull request #96 from ymaheshwari1/#2dmgzqb

Fixed: issue of temp expr not being set when scheduling a new job(#2dmgzqb)
hotwax · May 26, 2022 · c1d3d91 · c1d3d91
2 parents a5148e3 + c0e5810
commit c1d3d91
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/components/SaveThresholdModal.vue b/src/components/SaveThresholdModal.vue
@@ -181,10 +181,11 @@ export default defineComponent({
         'JOB_NAME': this.jobName ? this.jobName : this.userProfile.partyName,
         'SERVICE_NAME': job.serviceName,
         'SERVICE_COUNT': '0',
+        'SERVICE_TEMP_EXPR': 'EVERYDAY',
         'jobFields': {
           'productStoreId': productStoreId,
           'systemJobEnumId': job.systemJobEnumId,
-          'tempExprId': 'EVERYDAY',
+          'tempExprId': 'EVERYDAY',  // Need to remove this as we are passing frequency in SERVICE_TEMP_EXPR, currently kept it for backward compatibility
           'maxRecurrenceCount': '-1',
           'parentJobId': job.parentJobId,
           'runAsUser': 'system', // default system, but empty in run now

diff --git a/src/store/modules/job/actions.ts b/src/store/modules/job/actions.ts
@@ -308,10 +308,11 @@ const actions: ActionTree<JobState, RootState> = {
       'JOB_NAME': job.jobName,
       'SERVICE_NAME': job.serviceName,
       'SERVICE_COUNT': '0',
+      'SERVICE_TEMP_EXPR': job.jobStatus,
       'jobFields': {
         'productStoreId': this.state.user.currentEComStore.productStoreId,
         'systemJobEnumId': job.systemJobEnumId,
-        'tempExprId': job.jobStatus,
+        'tempExprId': job.jobStatus, // Need to remove this as we are passing frequency in SERVICE_TEMP_EXPR, currently kept it for backward compatibility
         'maxRecurrenceCount': '-1',
         'parentJobId': job.parentJobId,
         'runAsUser': 'system', // default system, but empty in run now