fix(ui): update computing unit status on /compute tab every second#4612
fix(ui): update computing unit status on /compute tab every second#4612gracecluvohio wants to merge 4 commits intoapache:mainfrom
Conversation
|
@aicam Please review it. |
| interval(1000) | ||
| .pipe(untilDestroyed(this)) | ||
| .subscribe(() => { | ||
| this.computingUnitStatusService.refreshComputingUnitList(); | ||
| }); |
There was a problem hiding this comment.
can we make it event driven, instead of every second?
There was a problem hiding this comment.
I think it would be a better user experience to make it every second, so that the status is as fresh as possible.
There was a problem hiding this comment.
we can deliver the same user experience. I am talking about implementation. can we make it event driven? for example, when some status changed, push an event to frontend to update the computing unit list.
There was a problem hiding this comment.
Unless I'm mistaken, I don't think the existing services we have expose "messages" or events sent from the backend, they just expose functionality for refreshing all of the computing units. I think a timer based refreshing is most consistent with the functions we currently have.
4 participants
What changes were proposed in this PR?
Refresh computing unit status on the
\computetab once per second.Demo:
Screen.Recording.2026-05-01.at.9.16.23.AM.mov
Any related issues, documentation, discussions?
Closes #4611
How was this PR tested?
Locally
Was this PR authored or co-authored using generative AI tooling?
No