This commit is contained in:
@@ -225,21 +225,19 @@ todo: many biz objects are not using new PUT methodology
|
|||||||
|
|
||||||
|
|
||||||
|
|
||||||
CURRENTLY DOING: error and ordering stuff
|
CURRENTLY DOING: implement actual delete of soft delete, if it works then port up to woitem and then strip out the old delete graph stuff
|
||||||
|
|
||||||
OVERALL
|
OVERALL
|
||||||
ORDERING AND ERRORS
|
ORDERING AND ERRORS
|
||||||
- DELETE ISSUE:
|
- DELETE ISSUE:
|
||||||
because it's removing the row on delete but not deleting until save, any errors can't be displayed correctly
|
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
|
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
|
|
||||||
or....maybe show them as read only grayed out??
|
|
||||||
if this, then delete would be wrong term, should be "mark for delete"
|
|
||||||
- on save iterate the graph looking for deleted=true and attempt delete
|
- 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
|
- 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
|
||||||
|
|
||||||
|
- Clean up save error handling code and remove comments
|
||||||
|
- clean up delete code in schedusers if not already
|
||||||
- test partial save (### and delete ###) with fail at each level (make up a fail if have to at server)
|
- 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
|
- Error messages should display correctly both in alert box and in each item's controls
|
||||||
- Test out role rights, login as various roles and ensure it works as expected
|
- Test out role rights, login as various roles and ensure it works as expected
|
||||||
@@ -251,6 +249,7 @@ OVERALL
|
|||||||
- ## Try to find every thing that will matter when more collections are added now, ensure a full flow from start to stop and everything in between to try to catch any gotchas now before get too deep into it ##
|
- ## Try to find every thing that will matter when more collections are added now, ensure a full flow from start to stop and everything in between to try to catch any gotchas now before get too deep into it ##
|
||||||
|
|
||||||
|
|
||||||
|
|
||||||
todo: WorkOrderItemTask??
|
todo: WorkOrderItemTask??
|
||||||
is this really a collection like scheduled users or labors now??
|
is this really a collection like scheduled users or labors now??
|
||||||
Does it show on a main table in it's own form??
|
Does it show on a main table in it's own form??
|
||||||
|
|||||||
@@ -487,7 +487,7 @@ export default {
|
|||||||
//handle errors
|
//handle errors
|
||||||
if (err.error) {
|
if (err.error) {
|
||||||
//# FAIL ROUTE
|
//# FAIL ROUTE
|
||||||
console.log("RAW ERROR OBJECT: ", err);
|
// console.log("RAW ERROR OBJECT: ", err);
|
||||||
//REHYDRATE ERRORS
|
//REHYDRATE ERRORS
|
||||||
//convert to form() error handling understood format
|
//convert to form() error handling understood format
|
||||||
let compiledError = {
|
let compiledError = {
|
||||||
@@ -521,7 +521,7 @@ export default {
|
|||||||
target: `Items[${z.woItemIndex}].scheduledUsers[${z.objectIndex}].${z.e.details[0].target}`
|
target: `Items[${z.woItemIndex}].scheduledUsers[${z.objectIndex}].${z.e.details[0].target}`
|
||||||
})
|
})
|
||||||
);
|
);
|
||||||
console.log("Compiled error ", compiledError);
|
//console.log("Compiled error ", compiledError);
|
||||||
vm.formState.serverError = compiledError;
|
vm.formState.serverError = compiledError;
|
||||||
window.$gz.form.setErrorBoxErrors(vm);
|
window.$gz.form.setErrorBoxErrors(vm);
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user