fix: format duration in backend

Author: wolfogre

routers/web/repo/actions/view.go

@@ -79,9 +79,9 @@ type ViewJob struct {
 }
 
 type ViewJobStep struct {
-	Summary  string  `json:"summary"`
-	Duration float64 `json:"duration"`
-	Status   string  `json:"status"`
+	Summary  string `json:"summary"`
+	Duration string `json:"duration"`
+	Status   string `json:"status"`
 }
 
 type ViewStepLog struct {
@@ -154,7 +154,7 @@ func ViewPost(ctx *context_module.Context) {
 		for i, v := range steps {
 			resp.StateData.CurrentJobSteps[i] = ViewJobStep{
 				Summary:  v.Name,
-				Duration: float64(v.Duration() / time.Second),
+				Duration: v.Duration().Round(time.Second).String(),
 				Status:   v.Status.String(),
 			}
 		}

web_src/js/components/RepoActionView.vue

@@ -51,7 +51,7 @@
               <SvgIcon name="octicon-x-circle-fill" class="red mr-3 " v-else/>
 
               <span class="step-summary-msg">{{ jobStep.summary }}</span>
-              <span class="step-summary-dur">{{ formatDuration(jobStep.duration) }}</span>
+              <span class="step-summary-dur">{{ jobStep.duration }}</span>
             </div>
 
             <!-- the log elements could be a lot, do not use v-if to destroy/reconstruct the DOM -->
@@ -162,27 +162,6 @@ const sfc = {
       });
     },
 
-    formatDuration(d) {
-      d = Math.round(d);
-      const unitValues = [60, 60, 24];
-      const unitNames = ['s', 'm', 'h', 'd'];
-      const parts = [];
-      for (let i = 0; i < unitValues.length; i++) {
-        parts[i] = d % unitValues[i];
-        d = Math.floor(d / unitValues[i]);
-      }
-      parts.push(d);
-      let res = '', resCount = 0;
-      for (let i = parts.length - 1; i >= 0 && resCount < 2; i--) {
-        if (parts[i] > 0) {
-          res += `${parts[i] + unitNames[i]} `;
-          resCount++;
-        }
-      }
-      if (!res) return '0s';
-      return res.substring(0, res.length - 1);
-    },
-
     createLogLine(line) {
       const div = document.createElement('div');
       div.classList.add('job-log-line');