Agent bugs and fixes (#1243)

* show Error for temp placeholder

* changed message to loading error

* changed error message when loading

* localized error msg

* localized error message
This commit is contained in:
Aditya Bist
2018-04-24 13:34:17 -07:00
committed by Karl Burtram
parent d0d4df313e
commit a2734807ca
2 changed files with 4 additions and 3 deletions

View File

@@ -1,6 +1,7 @@
// Adopted and converted to typescript from https://github.com/6pac/SlickGrid/blob/master/plugins/slick.rowdetailview.js
// heavily modified
import { mixin } from 'vs/base/common/objects';
import * as nls from 'vs/nls';
export class RowDetailView {
@@ -277,7 +278,7 @@ export class RowDetailView {
item._parent = parent;
item._offset = offset;
item.jobId = parent.jobId;
item.name = parent.message ? parent.message : 'Error';
item.name = parent.message ? parent.message : nls.localize('rowDetailView.loadError','Loading Error...');
return item;
}

View File

@@ -207,7 +207,7 @@ export class JobsViewComponent implements AfterContentChecked {
if (job.lastRunOutcome === 0 && !expandedJobs.get(job.jobId)) {
this.expandJobRowDetails(i+expandedJobs.size);
this.addToStyleHash(i+expandedJobs.size);
this._agentViewComponent.setExpanded(job.jobId, 'temp');
this._agentViewComponent.setExpanded(job.jobId, 'Loading Error...');
} else if (job.lastRunOutcome === 0 && expandedJobs.get(job.jobId)) {
this.expandJobRowDetails(i+expansions);
this.addToStyleHash(i+expansions);
@@ -298,7 +298,7 @@ export class JobsViewComponent implements AfterContentChecked {
let item = self.dataView.getItemById(job.jobId + '.error');
let noStepsMessage = nls.localize('jobsView.noSteps', 'No Steps available for this job.');
let errorMessage = jobHistory ? jobHistory.message: noStepsMessage;
item['name'] = item['name'] + ': ' + errorMessage;
item['name'] = nls.localize('jobsView.error', 'Error: ') + errorMessage;
self._agentViewComponent.setExpanded(job.jobId, errorMessage);
self.dataView.updateItem(job.jobId + '.error', item);