From af452f43e026797ef69ef6709b499f4a89df0113 Mon Sep 17 00:00:00 2001 From: John Cardinal Date: Fri, 15 Jan 2021 15:15:06 +0000 Subject: [PATCH] --- ayanova/devdocs/todo.txt | 11 +---------- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/ayanova/devdocs/todo.txt b/ayanova/devdocs/todo.txt index a8fc1f49..2959e539 100644 --- a/ayanova/devdocs/todo.txt +++ b/ayanova/devdocs/todo.txt @@ -5,19 +5,10 @@ MISC ITEMS THAT CAME UP -todo: PartAssemblyBiz shows the correct way to replace an object in it's PUT route - previously doing unnecessary copyobject to an interim snapshot object for no apparent reason - Go through all *biz classes and look to update to this method - also use GetAsync instead of directly querying again in the Put methods of all *biz objects where possible - and ensure it has AsNoTracking added because even when it's used in batch jobs etc they do a PUT and do not - directly save the retrieved object so that's what is appropriate todo: All indexes in db sb named starting with idx_ instead of just the table name, so just prepend idx_ to every index todo: foreign keys are not indexed by default in the referencing table (the one who references) take a look and see if any should be indexed when doing above -todo: All GetReportData in *Biz objects is not setting AsNoTracking which could result in huge overhead, add that to all -todo: *Biz ProcessBatchJobAsync is getting IDList without AsNoTracking, add it. -todo: All *Biz objects should use their own internal GetAsync rather than directly fetching the object to avoid multiple issues in future - (child collection properly handled, asnotracking properly handled etc) + todo: PickList controller route, how can I document the params into the api explorer? do I need a reference link to the class?