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

Fixes #36970 - Add ansible collections and other missing content counts to content counts #919

Merged
merged 1 commit into from
Jan 10, 2024

Conversation

ianballou
Copy link
Member

Adds more content types to hammer repository info.

Also ensures that blank content is hidden in hammer capsule content info

To test:

  1. Test hammer repository info and check that content counts are listed for all content types
  2. Quickly check that hammer capsule content info still shows counts correctly

@ianballou ianballou force-pushed the 36970-ac-content-counts branch 2 times, most recently from b5ded0d to 0756b9d Compare December 7, 2023 20:34
@ianballou ianballou force-pushed the 36970-ac-content-counts branch from 0756b9d to dc9dfa0 Compare January 8, 2024 21:35
Copy link
Member

@sjha4 sjha4 left a comment

Choose a reason for hiding this comment

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

Works well..Able to content counts of AC and other content types..Test failure seems unrelated. APJ 👍🏼

@ianballou
Copy link
Member Author

Merging, tests all pass locally at least.

@ianballou ianballou merged commit e09be6e into Katello:master Jan 10, 2024
2 of 3 checks passed
@ianballou ianballou deleted the 36970-ac-content-counts branch January 10, 2024 15:35
@chris1984
Copy link
Member

Ignore the test failure, will fix it at some point

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.

3 participants