department-of-veterans-affairs / va.gov-team

Public resources for building on and in support of VA.gov. Visit complete Knowledge Hub:
https://depo-platform-documentation.scrollhelp.site/index.html
275 stars 193 forks source link

Sidekiq: Remove Sidekiq polling dependency on front end submission. #83428

Open scottsdevelopment opened 2 weeks ago

scottsdevelopment commented 2 weeks ago

Per Sidekiq guidance, we should take extra caution and steps when dealing with dependencies with the front end. Such we have come to the conclusion that the best course of action would be to decouple the front end and the Sidekiq job completely. We had initially adopted a common pattern from the rest of the vets-website project that included this asynchronous polling of the job status, however the user confirmation page makes no use of the actual returned data and removing this polling would allow for an improvement to the confirmation page responsiveness and reduce overhead.

mjknight50 commented 2 days ago

After discussing this with @wayne-weibel and @scottsdevelopment , I think it is a good idea.

aplatt-coforma commented 2 days ago

Hey team! Please add your planning poker estimate with Zenhub @danlim715 @mjknight50 @scottsdevelopment @TaiWilkin @ToddWebDev @wayne-weibel