From 93b065217e2cf760a6807416ccf4d74a662e2e22 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sybren=20A=2E=20St=C3=BCvel?= Date: Tue, 12 Apr 2022 17:54:07 +0200 Subject: [PATCH] Web: always fetch fresh job on selection in the tabulator --- web/app/src/App.vue | 13 +------------ 1 file changed, 1 insertion(+), 12 deletions(-) diff --git a/web/app/src/App.vue b/web/app/src/App.vue index 32b84112..6f28702b 100644 --- a/web/app/src/App.vue +++ b/web/app/src/App.vue @@ -54,22 +54,11 @@ export default { methods: { // UI component event handlers: onSelectedJobChanged(jobSummary) { - const fullJob = API.Job.constructFromObject(jobSummary); - if (fullJob.created) { - // The 'created' field is not part of the JobUpdate, but it is part of - // Job, so if that field is there, we can assume that this is an - // already-fetched job. - // TODO: keep track of when this was fetched, and re-fetch if necessary. - this.selectedJob = fullJob; - return; - } - const jobsAPI = new API.JobsApi(this.apiClient); this._wrap(jobsAPI.fetchJob(jobSummary.id)) .then((job) => { this.selectedJob = job; - // Forward the full job to Tabulator, so that the next selection - // doesn't have to re-fetch it. + // Forward the full job to Tabulator, so that that gets updated too. this.$refs.jobsTable.processJobUpdate(job); }); },