fix(queuejobsbystate): bring logs only in job page

bee-queue · Mar 11, 2021 · 8ebd5c0 · 8ebd5c0
1 parent ca4fefc
commit 8ebd5c0
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 7 deletions.
diff --git a/src/server/views/dashboard/jobDetails.js b/src/server/views/dashboard/jobDetails.js
@@ -30,6 +30,11 @@ async function handler(req, res) {
   job.retryButtonText = jobState === 'failed' ? 'Retry' : 'Trigger';
   const stacktraces = queue.IS_BEE ? job.options.stacktraces : job.stacktrace;
 
+  if (!queue.IS_BEE) {
+    const logs = await queue.getJobLogs(job.id);
+    job.logs = logs.logs || 'No Logs';
+  }
+
   return res.render('dashboard/templates/jobDetails', {
     basePath,
     queueName,

diff --git a/src/server/views/dashboard/queueJobsByState.js b/src/server/views/dashboard/queueJobsByState.js
@@ -103,13 +103,6 @@ async function _html(req, res) {
   } else {
     const stateTypes = state === 'waiting' ? ['wait', 'paused'] : state;
     jobs = await queue.getJobs(stateTypes, startId, endId, order === 'asc');
-    await Promise.all(
-      jobs.map(async (job) => {
-        const logs = await queue.getJobLogs(job.id);
-        job.logs = logs.logs || 'No Logs';
-        return job;
-      })
-    );
   }
 
   for (const job of jobs) {