[CN-2583] add page refresh delay and resolve overlapping requests #25
+24
−25
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
arodski
changed the title
alice485
approved these changes
Sep 29, 2023
# for free
to join this conversation on GitHub.
Already have an account?
# to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Happy Path
Initial request made when pipeline is selected
2nd request made after 30s, response has timeout of 40s
On the next 30s interval, request is skipped due to request already in progress and waiting for response
2nd request completes before the next 30s interval
New request is made on the next 30s interval
Unhappy Path
Request made to refresh page and waits for response
Request fails, sets flag to false, and makes a 2nd request at the next 30s interval
Note: Utilized console logs for testing purposes.