Simplify final total calculation to prevent bugs (and solve collect_cash bug)
Analysis
With the new auto-approve logic, some final total calculations that are made in EnRouteToClientController should be moved into OrderFinalTotalRepo.
This will solve the collect_cash = 0 bug & make the code easier to read
Comments
Need to double check whether this would break for the type of orders that are not auto-approved. Unless we could just enable auto-approve for all orders
Business Value Added
Simplify final total calculation to prevent bugs (and solve collect_cash bug)
Analysis
With the new auto-approve logic, some final total calculations that are made in EnRouteToClientController should be moved into OrderFinalTotalRepo. This will solve the collect_cash = 0 bug & make the code easier to read
Comments
Need to double check whether this would break for the type of orders that are not auto-approved. Unless we could just enable auto-approve for all orders