fecgov / fecfile-web-api

Back-end API for FECfile application
7 stars 2 forks source link

Performance Improvements - Report Recalculation Flagging #917

Open Elaine-Krauss-TCG opened 2 weeks ago

Elaine-Krauss-TCG commented 2 weeks ago

When flagging reports for recalculation, individual reports are instantiated just to update a single field, and this has an outsized impact on the response time for creating transactions. Instead of instantiating each report, we should instead call .update() on the filtered queryset to change just the field in question.

QA Notes

After creating or updating a transaction, when you go to a different report with coverage dates later in the same year, the summary page should need to recalculate.

DEV Notes

null

Design

null

FECFILE-1429

mjtravers commented 1 week ago

Passes CR. Sending to QA.

WiseQA commented 1 week ago

QA review verified with DEV visual inspection of the Performance Improvement.

QA Review Completed. Moved to Stage Ready.