Skip to content

Conversation

@vishesh92
Copy link
Member

Description

This PR adds minor fixes on #12552

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (improves an existing feature and functionality)
  • Cleanup (Code refactoring and cleanup, that may add test cases)
  • Build/CI
  • Test (unit or integration test code)

Feature/Enhancement Scale or Bug Severity

Feature/Enhancement Scale

  • Major
  • Minor

Bug Severity

  • BLOCKER
  • Critical
  • Major
  • Minor
  • Trivial

Screenshots (if appropriate):

How Has This Been Tested?

How did you try to break this feature and the system with this change?

@vishesh92
Copy link
Member Author

@blueorangutan ui

@blueorangutan
Copy link

@vishesh92 a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

@codecov
Copy link

codecov bot commented Jan 30, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 16.26%. Comparing base (45d623e) to head (67c6bfa).
⚠️ Report is 2 commits behind head on 4.20.

Additional details and impacted files
@@             Coverage Diff              @@
##              4.20   #12553       +/-   ##
============================================
+ Coverage     4.16%   16.26%   +12.09%     
- Complexity       0    13427    +13427     
============================================
  Files          403     5660     +5257     
  Lines        32905   499917   +467012     
  Branches      5881    60698    +54817     
============================================
+ Hits          1371    81318    +79947     
- Misses       31360   409525   +378165     
- Partials       174     9074     +8900     
Flag Coverage Δ
uitests 4.16% <ø> (-0.01%) ⬇️
unittests 17.11% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@blueorangutan
Copy link

UI build: ✔️
Live QA URL: https://qa.cloudstack.cloud/simulator/pr/12553 (QA-JID-863)

@sureshanaparti sureshanaparti added this to the 4.20.3 milestone Jan 30, 2026
@sureshanaparti
Copy link
Contributor

Verified the filter issues - mainly in Hosts view (displaying Type= in fliter) and filter uuid displayed instead of name on scrolling the view.

@vishesh92 vishesh92 force-pushed the ui-search-filter-prevent-extra-call branch from c27ccb9 to 67c6bfa Compare January 30, 2026 08:31
@vishesh92
Copy link
Member Author

@blueorangutan ui

@blueorangutan
Copy link

@vishesh92 a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

Copy link
Contributor

@DaanHoogland DaanHoogland left a comment

Choose a reason for hiding this comment

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

lgtm

@vishesh92 vishesh92 mentioned this pull request Jan 30, 2026
14 tasks
@blueorangutan
Copy link

UI build: ✔️
Live QA URL: https://qa.cloudstack.cloud/simulator/pr/12553 (QA-JID-864)

@DaanHoogland DaanHoogland merged commit d0e21a7 into apache:4.20 Jan 30, 2026
26 checks passed
@DaanHoogland DaanHoogland deleted the ui-search-filter-prevent-extra-call branch January 30, 2026 08:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants