Added Smart Auto-Refresh System to Dashboard Headerauto refresh added#156
Added Smart Auto-Refresh System to Dashboard Headerauto refresh added#156Himanshityagii24 wants to merge 1 commit intovolcano-sh:mainfrom
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
Signed-off-by: Himanshityagii24 <tyagihimanshi241@gmail.com>
d197f65 to
4eb44b0
Compare
|
/assign @Monokaix |
|
I have already implemented this in PR/148. |
|
@karanBRAVO That's excellent! SSE and client-side auto-refresh actually complement each other perfectly - SSE handles real-time server-initiated updates while auto-refresh gives users control over their monitoring frequency and provides a reliable fallback when SSE connections drop.Rather than competing approaches, this creates a robust hybrid system where users get both real-time updates AND the flexibility to control their data refresh experience. |
|
But that PR is already having refresh button :) |
|
Yes!,but users still need manual control for specific monitoring scenarios |
|
SSE connections often fail, and during those times, this hybrid system can be very helpful. So why stick with a manual refresh button? |
|
Ok, that's correct, but you can create a separate settings page for this kind of stuff |
|
Yeah, that is something I will propose as well. First, let me get approval for the idea |
Description
Importance
Screenshots