devdocs/todo.txt

@@ -8,7 +8,13 @@ PRIORITY - ALWAYS Lowest level stuff first
 
 
 
-todo: Licensing - make sure tech type use license works properly: a tech license is consumed if a user has **any one** of the four roles of TechFull, TechLimited, SubcontractorFull, SubcontractorLimited
+todo: Licensing - make sure tech type use license works properly: a tech license is consumed if a user has 
+**any one** of the four roles of TechFull, TechLimited, SubcontractorFull, SubcontractorLimited
+	NO, NOT TRUE AT ALL, those are authorization roles, not user types, it's coded properly already
+todo: usertype scheduleable?  Or sb Service and nonScheduleble sb NonService and subcontractor sb ServiceContractor?
+	will require some renaming of things but this is more logical and meets intended usage
+todo: Pick a consistent single letter lambda expression variable and standardize on it.
+	one of X Y or Z is best as it doesn't trigger intellisense
 todo: OPS routes (SERVER AND CLIENT)
 	- Backup, restore https://rockfish.ayanova.com/default.htm#!/rfcaseEdit/3369
 	- These need to be done fairly early on in order to have shit to play with for testing etc

server/AyaNova/biz/UserBiz.cs

@@ -32,7 +32,9 @@ namespace AyaNova.Biz
         internal static async Task<long> ActiveCountAsync()
         {
             var ct = ServiceProviderProvider.DBContext;
-            var ret = await ct.User.Where(x => x.Active == true && (x.UserType == UserType.Schedulable || x.UserType == UserType.Subcontractor)).LongCountAsync();
+            var ret = await ct.User.Where(x => x.Active == true && (            
+                x.UserType == UserType.Schedulable || 
+                x.UserType == UserType.Subcontractor)).LongCountAsync();
 
             return ret;
         }