Skip to content

Fix excessive Detailed Results updates

Placeholder Jimmy Keith requested to merge 353_fix_detailedresult_callcount into develop

@ mention of reviewers`

@ckcollab`

A brief description of the purpose of the changes contained in this PR.

Only updates the detailed results URL after all the lines of the catchup message have been addressed, so we only use the final URL. Should limit the number of detail_result.html calls from the frontend and speed up page refresh

Issues this PR resolves

resolves #353 (closed)

A checklist for hand testing

comp URL: http://134.158.74.226/competitions/53/#/participate-tab can download one of the already finished submissions from the submission table.

  • run submission
  • wait for the graph to be updated a few times (so there are multiple detail_result updates in the tmp/.txt file
  • on refresh, should only be a few detailed results update calls (like 5ish depending on the timing of your refresh), instead of many

Checklist

  • Code review by me
  • Hand tested by me
  • I'm proud of my work
  • Code review by reviewer
  • Hand tested by reviewer
  • Ready to merge

Merge request reports