Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(RHINENG-13687): convert radio filter to single select #1239

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

johnsonm325
Copy link
Contributor

@johnsonm325 johnsonm325 commented Jan 10, 2025

This PR leverages the new conditional filter type singleSelect to convert radio dropdown filters to single select dropdown filters. These filters can be found on Advisories page -> Publish date filter and Systems page -> Updatable filter.

@johnsonm325 johnsonm325 added the don't merge This pull request shouldn't be merged yet label Jan 10, 2025
@johnsonm325 johnsonm325 requested a review from a team as a code owner January 10, 2025 17:08
@johnsonm325 johnsonm325 marked this pull request as draft January 10, 2025 17:08
@johnsonm325 johnsonm325 changed the title DRAFT: fix(RHINENG-13687) fix(RHINENG-13687): convert radio filter to single select Jan 23, 2025
@johnsonm325 johnsonm325 marked this pull request as ready for review January 23, 2025 16:51
@johnsonm325 johnsonm325 removed the don't merge This pull request shouldn't be merged yet label Jan 23, 2025
@codecov-commenter
Copy link

codecov-commenter commented Jan 23, 2025

Codecov Report

Attention: Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 63.59%. Comparing base (552e67d) to head (682105a).

Files with missing lines Patch % Lines
src/Utilities/Helpers.js 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1239      +/-   ##
==========================================
- Coverage   63.60%   63.59%   -0.01%     
==========================================
  Files         124      124              
  Lines        3267     3269       +2     
  Branches      860      860              
==========================================
+ Hits         2078     2079       +1     
- Misses       1189     1190       +1     

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

@gkarat gkarat requested a review from a team January 24, 2025 13:03
@gkarat gkarat added the enhancement New feature or request label Jan 24, 2025
@johnsonm325 johnsonm325 force-pushed the rhineng-13687 branch 2 times, most recently from 0e547ea to 37b0b1e Compare January 24, 2025 15:41
@LightOfHeaven1994
Copy link
Contributor

@johnsonm325 we get Error: Invalid conditional filter component type! Expected one of text,checkbox,radio,custom,group, got singleSelect on Systems page for some reason. But it works on Advisory page

This PR leverages the new conditional filter type singleSelect to
convert radio dropdown filters to single select dropdown filters. These
filters can be found on Advisories page -> Publish date filter and
Systems page -> Updatable filter.
@johnsonm325
Copy link
Contributor Author

@LightOfHeaven1994 The Systems table filter should work now. The PR to fix the Inventory Table has been merged: RedHatInsights/insights-inventory-frontend#2337.

And the release to production was done today.

@johnsonm325
Copy link
Contributor Author

/retest

2 similar comments
@johnsonm325
Copy link
Contributor Author

/retest

@johnsonm325
Copy link
Contributor Author

/retest

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants