From 32cc6ac5de6a123f15840ac405c1a8e5888de8cd Mon Sep 17 00:00:00 2001 From: John Cardinal Date: Wed, 13 May 2020 00:09:44 +0000 Subject: [PATCH] --- devdocs/todo.txt | 21 ++++++++++++++------- 1 file changed, 14 insertions(+), 7 deletions(-) diff --git a/devdocs/todo.txt b/devdocs/todo.txt index f8aacc0a..698e251e 100644 --- a/devdocs/todo.txt +++ b/devdocs/todo.txt @@ -1,5 +1,7 @@ -todo: add query fail logging to datalist just like done with picklist so in production can catch mysterious problems more easily +PRIORITY - ALWAYS Lowest level stuff first +=-=-=-=- +TODO: check non-ops user can login during seeding operation TODO: BizRoles.cs seems to get hammered on every single request, is it efficient? - Why is it not cached in some way? - Look into Lazy initialization (i.e. private static readonly Lazy> mappings ....etc) @@ -7,10 +9,7 @@ TODO: BizRoles.cs seems to get hammered on every single request, is it efficient if do so then ideally put them into the biz object responsible? - it has to instantiate that anyway on any meaningful operation or...is this worse than actually caching in a lifetime cache because it will need to load on every route hit whereas a lifteimte cache is just there from being loaded once? -todo: AUTO ID GENERATOR - change to a dedicated spot in global rather than inferring as it is not right practically for reasons - Get the auto-id most recent values at boot time - internal static void SetMostRecentAutoIdValuesFromDatabase(AyaNova.Models.AyContext ct) + TODO: API REFACTORING (note: workordercontroller / biz should be following all these rules so it's the template if need reference) todo: consider and then get rid of PATCH in all routes unless compelling reason to keep @@ -31,6 +30,13 @@ todo: Routes should check rights *BEFORE* they fetch the object, not after, all i.e. delete route instantiates biz object, then it fetchs object from db *then* it checks if they have rights to delete (generically, not specific to that object) This is out of order as it triggers a db call even if they have no rights to do it todo: all biz objects "ExistsAsync" is this required / necessary? + +todo: add query fail logging to datalist just like done with picklist so in production can catch mysterious problems more easily +todo: AUTO ID GENERATOR + change to a dedicated spot in global rather than inferring as it is not right practically for reasons + Get the auto-id most recent values at boot time + internal static void SetMostRecentAutoIdValuesFromDatabase(AyaNova.Models.AyContext ct) + todo: Search get search result for workorder descendents doesn't fit into existing system due to it being a biz object wide feature but workorder having sub-get results - CHILD objects need to point to their parent and be readable in code for searching and for opening objects based on child object - so all child objects need a typeandid of the immediate parent @@ -58,8 +64,8 @@ todo: (BREAK THIS OUT INTO LATER/NOW/CASES) there are several outstanding AUTHEN https://rockfish.ayanova.com/default.htm#!/rfcaseEdit/3250 todo: Look into 2fa https://rockfish.ayanova.com/default.htm#!/rfcaseEdit/3395 - -TODO: check non-ops user can login during seeding operation + + todo: Log configuration into server log at startup? https://rockfish.ayanova.com/default.htm#!/rfcaseEdit/3523 log all config settings into server log on boot @@ -71,6 +77,7 @@ TODO: Licensing - make sure tech type use license works properly: a tech license TODO: API docs, make separate page for datalists and remove from api-response-format.md doc but put a reference link to it there. + ------------------------------- ### LATER (TURN INTO CASES IN ROCKFISH WHERE POSSIBLE AND REMOVE FROM HERE)