From 3afdc2015548f0a67a3867e7061d35f4f7a9c39f Mon Sep 17 00:00:00 2001 From: John Cardinal Date: Wed, 9 Mar 2022 18:04:21 +0000 Subject: [PATCH] Fixed bug updating status breaks concurrency token for header so can't update it --- ayanova/devdocs/todo.txt | 13 +++++-------- ayanova/src/views/svc-quote.vue | 2 ++ ayanova/src/views/svc-workorder.vue | 3 +++ 3 files changed, 10 insertions(+), 8 deletions(-) diff --git a/ayanova/devdocs/todo.txt b/ayanova/devdocs/todo.txt index 4835b9e8..e626dd95 100644 --- a/ayanova/devdocs/todo.txt +++ b/ayanova/devdocs/todo.txt @@ -23,13 +23,6 @@ FIRST CLIENT SOURCE CODE COMMIT JAN 3rd 2019 # OUTSTANDING MAJOR AREAS TO BETA -test user color in home user and in adm-user with new record and exsiting just to confirm it's ok -can something be done with usercolor in schedule team view?? - - - -bugbug: change status on quote and save then can't change other things and save. Maybe same with workorder too. - because last status being set on header and not updating it and concurrency on save?? bugbug: home-notifications text in general error is cut off, doesn't overflow properly just goes off the screen as if not contained seems like static pre text so maybe throw it in a div or add an overflow-auto to it or something @@ -205,6 +198,9 @@ todo:3 todo: check out this SPA memory leak detector, it *does* have a login abi todo: 1 should a superuser really have a dashboard? +todo: 2 or 3 category team view color based on user color? + check category schedule control docs or implement in my slot override + todo: 2 create workorder, quote, pm from customer record?? was that a v7 feature? make it so? v.next? @@ -822,4 +818,5 @@ BUILD 8.0.0-beta.1-rc4 CHANGES OF NOTE - Added CustomerNotification feature, exposed as Customer notifications in UI Customers section, see corresponding help doc for details - Added user notification delivery log in OPS - Added Customer notification delivery log in OPS -- Updated notification types doc page, was missing a few, also sorted alphabetically was random chaos order \ No newline at end of file +- Updated notification types doc page, was missing a few, also sorted alphabetically was random chaos order +- Change to ensure color picker shows Alpha selector to control opacity \ No newline at end of file diff --git a/ayanova/src/views/svc-quote.vue b/ayanova/src/views/svc-quote.vue index 84174cf2..de311a63 100644 --- a/ayanova/src/views/svc-quote.vue +++ b/ayanova/src/views/svc-quote.vue @@ -854,6 +854,8 @@ async function saveState(vm) { handleSaveError(vm, { error: res.error }); } else { vm.obj.states[i] = res.data; + //changing status updates the quote header as well so need new concurrency token + vm.obj.concurrency = res.data.newQuoteConcurrency; //set locked status of entire wo now vm.obj.isLockedAtServer = vm.currentState.locked; diff --git a/ayanova/src/views/svc-workorder.vue b/ayanova/src/views/svc-workorder.vue index 0df000e6..608c1b3e 100644 --- a/ayanova/src/views/svc-workorder.vue +++ b/ayanova/src/views/svc-workorder.vue @@ -945,6 +945,9 @@ async function saveState(vm) { } else { vm.obj.states[i] = res.data; + //changing status updates the laststatus in the wo header so need to update concurrency here as well + vm.obj.concurrency = res.data.newWOConcurrency; + //set locked status of entire wo now vm.obj.isLockedAtServer = vm.currentState.locked; }