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 #36782 - Update UI in preparation for SCA-only #10753

Merged
merged 1 commit into from
Sep 29, 2023

Conversation

jeremylenz
Copy link
Member

@jeremylenz jeremylenz commented Sep 26, 2023

What are the changes introduced in this pull request?

Add UI messaging in preparation for the retirement of entitlement mode in Katello 4.12.

image

image

image

image

image

Considerations taken when implementing this change?

Once the branding plugin is updated, "Katello 4.12" will translate to "Satellite 6.16" downstream.

What are the testing steps for this pull request?

Check all of the following locations:

  • Non-SCA Banner messages updated with specific Katello version (4.12) on
    • Subscriptions page
    • Legacy Content Host details page
    • Activation Key details page
  • Organization Edit screen has a message next to the SCA checkbox
  • SCA Banner message (Subscriptions service link) updated per UX guidance:
  • For SCA organizations, banner will be removed
  • The second sentence ("Learn more about your subscription usage..."), with the link to the Subscriptions service, will appear inside a Popover. This popover will be accessed by clicking a question mark icon (?) to the right of the "Subscriptions" title/header.
  • The question mark icon / popover is not visible for organizations with no manifest imported.
  • Also, adjusted some wording for Subscriptions page empty states.

@theforeman-bot
Copy link

Issues: #36782

@jeremylenz jeremylenz force-pushed the 36782-sca-only-banners branch from 467495b to 9583db7 Compare September 27, 2023 13:06
@jeremylenz jeremylenz marked this pull request as ready for review September 27, 2023 13:07
@jeremylenz jeremylenz force-pushed the 36782-sca-only-banners branch from 9583db7 to 3aab7a1 Compare September 27, 2023 13:32
Copy link
Member

@lfu lfu left a comment

Choose a reason for hiding this comment

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

This worked great! Nice job!

@jeremylenz jeremylenz merged commit dbfa00b into Katello:master Sep 29, 2023
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.

3 participants