Skip to content

UI temp remove polling#1722

Merged
cmp5987 merged 3 commits intomainfrom
ui-temp-remove-polling
Feb 5, 2026
Merged

UI temp remove polling#1722
cmp5987 merged 3 commits intomainfrom
ui-temp-remove-polling

Conversation

@cmp5987
Copy link
Collaborator

@cmp5987 cmp5987 commented Feb 5, 2026

What type of PR is this?

/kind bug

What this PR does / why we need it:

  • Disables 'Next' and 'Prev' Button when loading more data
  • moves loading state of table to only occur on initialload
  • Adds storing, memorization of data
  • Removes polling because the behavior wasn't desired.

Addresses flashing loading state when users change filters/sorting/page of primary views.

@cmp5987 cmp5987 requested a review from hulto February 5, 2026 01:40
@cmp5987 cmp5987 added bug Something isn't working Frontend labels Feb 5, 2026
@cmp5987 cmp5987 force-pushed the ui-temp-remove-polling branch from 71b6d6f to 1769ed3 Compare February 5, 2026 01:44
@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2026

Summary

Tests 📝 Passed ✅ Failed ❌ Skipped ⏭️ Other ❓ Flaky 🍂 Duration ⏱️
2544    ±0 2544    ±0 0    ±0 0    ±0 0    ±0 0    ±0 1ms    ±0

Previous Results

Build 🏗️ Result 🧪 Tests 📝 Passed ✅ Failed ❌ Skipped ⏭️ Other ❓ Flaky 🍂 Duration ⏱️
#813 2544 2544 0 0 0 0 15.3s

Insights

Average Tests per Run Total Flaky Tests Total Failed Slowest Test (p95)
2544 0 0 5.2s

Slowest Tests

Test 📝 Results 📊 Duration (avg) ⏱️ Duration (p95) ⏱️
eldritch-libsys: std::dll_inject_impl::tests::test_dll_inject_simple 1 5.2s 5.2s
imix::bin/imix: install::tests::test_install_execution 3 1.2s 3.4s
imix::bin/imix: install::tests::test_install_execution 3 1.2s 3.4s
imix::bin/imix: install::tests::test_install_execution 3 1.2s 3.4s
imix::bin/imix: tests::task_tests::test_task_streaming_error 3 3.0s 3.0s
imix::bin/imix: tests::task_tests::test_task_streaming_error 3 3.0s 3.0s
imix::bin/imix: tests::task_tests::test_task_streaming_error 3 3.0s 3.0s
imix::bin/imix: tests::task_tests::test_task_streaming_output 3 3.0s 3.0s
imix::bin/imix: tests::task_tests::test_task_streaming_output 3 3.0s 3.0s
imix::bin/imix: tests::task_tests::test_task_streaming_output 3 3.0s 3.0s

🎉 No failed tests in this run. | 🍂 No flaky tests in this run.

Github Test Reporter by CTRF 💚

🔄 This comment has been updated

@cmp5987 cmp5987 merged commit 95c1896 into main Feb 5, 2026
8 checks passed
@cmp5987 cmp5987 deleted the ui-temp-remove-polling branch February 5, 2026 02:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working Frontend

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants