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(viewToolbar): dsc-682 add missing types to filter #370

Merged
merged 1 commit into from
May 15, 2024

Conversation

mirekdlugosz
Copy link
Contributor

Credential type and Source type filter items were missing RHACS and Ansible. We must have forgot to add them back when we introduced these source types. Add these items now.

Relates to JIRA: DISCOVERY-682

Credential type and Source type filter items were missing RHACS and
Ansible. Add them.

Relates to JIRA: DISCOVERY-682
https://issues.redhat.com/browse/DISCOVERY-682
@mirekdlugosz mirekdlugosz requested review from cdcabrera and a team May 14, 2024 08:32
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.24%. Comparing base (3d66168) to head (2d076ce).
Report is 23 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #370      +/-   ##
==========================================
+ Coverage   84.19%   84.24%   +0.05%     
==========================================
  Files         116      116              
  Lines        3783     3765      -18     
  Branches     1345     1341       -4     
==========================================
- Hits         3185     3172      -13     
+ Misses        537      533       -4     
+ Partials       61       60       -1     
Files Coverage Δ
src/components/viewToolbar/viewToolbarSelect.js 100.00% <100.00%> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 91dd7eb...2d076ce. Read the comment docs.

Copy link
Member

@cdcabrera cdcabrera left a comment

Choose a reason for hiding this comment

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

lgtm

@mirekdlugosz mirekdlugosz merged commit 6863500 into quipucords:main May 15, 2024
3 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants