department-of-veterans-affairs / va-mobile-app

"If VA were a company, it would have a flagship mobile app."
https://department-of-veterans-affairs.github.io/va-mobile-app/
17 stars 2 forks source link

CU - Spike - Review Maint Window Code for efficiency/reporting improvements #7337

Open dumathane opened 12 months ago

dumathane commented 12 months ago

Proposed Change

Our current maintenance windows are leaving things to be desired, allowing lots of API calls to continue being called despite the user being inside a downtime window. We should also confirm whether or not an refresh timer makes sense for this as well considering it is only ever queried at sync screen.

Why Should We Prioritize?

Maint Windows are in place for a reason and we should endeavor to limit mistaken/unusable calls as much as possible.

The following are api calls that took place recently despite the COLA maint window: image

Coding Time Estimation

5

Testing Considerations

Checklist

[ ] Add the front-end label tag

[ ] Attach to the Frontend epic for engineering initiatives

timwright12 commented 3 weeks ago

@dumathane @ala-yna what's the status of this ticket? That time of year is coming up again and we're anticipating another downtime window

ala-yna commented 3 weeks ago

Hasn't been prioritized yet, but I think we could work it in.

lexicalninja commented 2 days ago

@timwright12 @dumathane is there someone to run through this with me so that I can either take a stab at it or spike it?

dumathane commented 2 days ago

I chatted with Patrick about this one and am looking to basically get the same report after this years COLA downtime to compare and identify any next steps necessary.