Skip to content
This repository has been archived by the owner on Dec 20, 2023. It is now read-only.

fix: Recent test runs page does not show new test runs unless page is… #20

Merged
merged 1 commit into from
Feb 27, 2022
Merged

fix: Recent test runs page does not show new test runs unless page is… #20

merged 1 commit into from
Feb 27, 2022

Conversation

madhavsikka
Copy link
Contributor

Closes #19

Description

  • Fixes #19
  • Added polling to fetch updated test runs periodically.

Additional Information

  • Polling duration has been set to 500ms

@madhavsikka
Copy link
Contributor Author

Need some feedback on polling duration ? Does 500ms look good ?

@nehagup
Copy link
Member

nehagup commented Feb 27, 2022 •
edited
Loading

Thanks for the Pull Request @madhavsikka
I shall review this soon.

Need some feedback on polling duration ? Does 500ms look good ?

Yes it seems good to me.

@nehagup nehagup self-requested a review February 27, 2022 05:17
nehagup
nehagup approved these changes Feb 27, 2022
View reviewed changes
Copy link
Member

@nehagup nehagup left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works like a charm! 🌟

@nehagup nehagup merged commit 82ab6c5 into keploy:main Feb 27, 2022
@madhavsikka madhavsikka deleted the query-polling branch February 27, 2022 09:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@nehagup nehagup nehagup approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[bug]: Recent test runs page does not show new test runs unless page is refreshed
2 participants
@madhavsikka @nehagup