ayanova/devdocs/todo.txt

@@ -315,6 +315,20 @@ todo: contract changes
 
 todo: test contract changing, application on new etc and fully works with labor before getting too carried away carrying over to other things
 
+todo: tax code ref integrity check should include wo or...:
+todo: tax codes, should they just be used to SET a mirror set of fields in other objects thus allowing them to be deleted etc`
+	currently the workorder stores the tax code id as well as all the data about it that is relevant to teh workorder, but this sets up a link between them
+	that is technically unnecessary.  If the intention is to decouple them, this is a bit of a wasted redundancy, however it does flow like all other things cleanly and displays etc
+	not sure what to do with this, needs a think.  
+
+	It *is* important to keep the redundancy, that's a good solid new feature...hmmm...
+	perhaps the issue is if they delete the tax code it should set the woitem* things that store it's id to null as it will no longer exist, but then the name wouldn't show
+
+	I'd like to see it implemented as somethign that is just applied to it and not something that is stored with it and flow that out to PO and global settings etc
+	then it's less sticky, people can just delete a tax code any time and don't need to rule check all the shit
+
+	Circle back to this one
+
 ORDER: Travel, tasks, parts, loans, units, outside service
 
 todo: "DispatchFull" and "DispatchLimited" roles MUST be renamed to "ServiceFull" and "ServiceLimited" to match other roles and intent