GSA / notifications-admin

The UI of Notify.gov
https://notify.gov
Other
11 stars 2 forks source link

Address instability from report download caching #2094

Open ecayer opened 2 weeks ago

ecayer commented 2 weeks ago

In order to improve report performance, the team initially implemented a cache, but this isn't working. We are going to replace the cache with an approach that separates out report generation from downloading in the UI.

For more details on the problem and proposed approach, see here the ADR to Improve API stability when processing job data and generating reports

A full fix for this includes the following fixes and improvements and investigations:

ecayer commented 2 weeks ago

@ccostino Do you have a preference whether we group the work to decouple report generation/download with this or track it in a separate epic? And there isn't yet an issue that includes the UI for that, right?

ccostino commented 1 week ago

@ecayer I think this does need to be split up now at this point... how does this sound?

For the UI side of all of this you're right, there is no issue yet per se but it's been loosely captured in the ADR. There's been some conversation that's mostly led to more questions and the need for at least some user research.

To me that sounds like a new issue entirely for that, but I figured I'd gut check to be sure! Regardless, if this makes sense, I can help write up some more of the details to help get things more organized with this!