Problem
We are inconsistent with what report total we use in the report action buttons in the workspace chat when the user can, for example, submit a report of a certain value.
Solution
Let's clean this up to achieve this set-up:
- Submit: use overall report total
- Review: use overall report total
- Approve: use overall report total
- Pay: only use the amount that would actually be paid
- Export: we don't use the total here
Let's abstract this logic into a separate method and cover its logic with automated tests.
Note this means we would also include the held items values in the overall report total - as we already show the hold modal for the users to confirm before action is actually completed.
Issue Owner
Current Issue Owner: @JakubKorytko
Problem
We are inconsistent with what report total we use in the report action buttons in the workspace chat when the user can, for example, submit a report of a certain value.
Solution
Let's clean this up to achieve this set-up:
Let's abstract this logic into a separate method and cover its logic with automated tests.
Note this means we would also include the held items values in the overall report total - as we already show the hold modal for the users to confirm before action is actually completed.
Issue Owner
Current Issue Owner: @JakubKorytko