From 0caa09b763d2ec4d9068d4ad56234406969602b0 Mon Sep 17 00:00:00 2001 From: John Cardinal Date: Thu, 15 Apr 2021 23:22:43 +0000 Subject: [PATCH] --- ayanova/devdocs/todo.txt | 18 +++++++----------- 1 file changed, 7 insertions(+), 11 deletions(-) diff --git a/ayanova/devdocs/todo.txt b/ayanova/devdocs/todo.txt index 9ae3f8a3..6434fa9e 100644 --- a/ayanova/devdocs/todo.txt +++ b/ayanova/devdocs/todo.txt @@ -229,17 +229,13 @@ CURRENTLY DOING: error and ordering stuff OVERALL ORDERING AND ERRORS - - since each graph item is saved one by one, the client knows which one it's sending so the Client should hydrate the correct collection error index - server should act like its' a standalone object when reporting errors - this way, no need to modify existing code at all (po etc) - - woitem must show error indicator in row if error is in it's children so user knows where to go - - - `Items[${activeWoItemIndex}].scheduledUsers[ - ${activeItemIndex} - ].estimatedQuantity` - Items[3].scheduledUsers[ 0 ].estimatedQuantity - target: "Items[3].scheduledUsers[0].EstimatedQuantity" + - DELETE ISSUE: + because it's removing the row on delete but not deleting until save, any errors can't be displayed correctly + TODO: move to a "soft delete" get rid of the deleted items separate array + - on delete flag an item as "deleted=true" + - modify the Items method that provides the array to the table so hide "deleted=true" items + - on save iterate the graph looking for deleted=true and attempt delete + - on fail of delete set deleted=true to false so it will show but make sure to do it *IN* the delete method so it's ready for later to show the error - test partial save (### and delete ###) with fail at each level (make up a fail if have to at server) - Error messages should display correctly both in alert box and in each item's controls