This commit is contained in:
@@ -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?
|
||||
|
||||
Reference in New Issue
Block a user