Problem
The report preview action buttons need transaction violations to decide what to show.
Right now this is computed more than once instead of one shared place. Computing
violations is not free, so doing it per-button adds extra work on each re-render.
Goal
- Compute the transaction violations one time in
MoneyRequestReportPreviewProvider.
- Share the result through a context slice.
- Update the action buttons to read violations from context instead of computing
them on their own.
- Measure the performance gain first, and add the numbers to the PR so we can
confirm the change is worth it.
Related
Issue Owner
Current Issue Owner: @linhvovan29546
Problem
The report preview action buttons need transaction violations to decide what to show.
Right now this is computed more than once instead of one shared place. Computing
violations is not free, so doing it per-button adds extra work on each re-render.
Goal
MoneyRequestReportPreviewProvider.them on their own.
confirm the change is worth it.
Related
Issue Owner
Current Issue Owner: @linhvovan29546