-
Notifications
You must be signed in to change notification settings - Fork 296
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 #36492 - duplicate content warning for composite CVs for non-docker repos #10741
Conversation
Issues: #36492 |
Test failures expected. Will update those. |
4aee0d4
to
7644c2a
Compare
Updated tests..Should be green.. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't use the term "component content views" any more..
7644c2a
to
d1b45c6
Compare
d1b45c6
to
7bb76f2
Compare
7bb76f2
to
d689532
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works as expected 👍
APJ
What are the changes introduced in this pull request?
Adds a notice to composite CVs publish screen if component CV versions have the same repositories added.
Considerations taken when implementing this change?
What are the testing steps for this pull request?
1, Create 2 CV versions with same repo
2.Add both to a composite CV
3. Publish. You should see a notice explaining duplicate content will be merged into the Composite CV version. The text is copied from earlier: https://github.com/Katello/katello/pull/8198/files