Data from submit/authorize/approve/skip/reject endpoint minimized by using a basic dto - this should impact UI calculations after some small changes, so we decide that we can merge these changes.
This does not change much with the UI, except the offline requisition will no longer be overwritten when changing status
There is some concern about the UI calculations not beiing sent to the backend for validation (the calculated fields are getting nulled) - but this commit does not really change things with this, currently the user is shown the UI calculations as well.
We weren't sure if backend validates calculation, bust likely so - if this is the case, the UI should not set calculated values to null before sending. This would make us confident in the UI calculations.
Non full supply products
Non full supply products are now fetched with a single call instead of looping through them and fetching them one by one - this is the only change to them that was made (except the above response trimming). Apparently the performance gain is from 30s to 5s.
The above change is a PR from Jakub Kondrat - he is doing last fixes for it
Caching the orderables might be a future improvement
Malawi performance tickets are outdated and do not reflect the work being done in them
Nikodem Graczewski (Unlicensed) to create a ticket for making the UI send calculated values to the backendd, if the latter support validating these calculations (we believe it should)