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

feat: update warning messages by auth method #4208

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

Conversation

DuckBoss
Copy link
Contributor

@DuckBoss DuckBoss commented Sep 9, 2024

All Pull Requests:

Check all that apply:

  • Have you followed the guidelines in our Contributing document, including the instructions about commit messages?
  • Is this PR to correct an issue?
  • Is this PR an enhancement?

Complete Description of Additions/Changes:

This patch updates the messages shown to the user
when the machine is registered/not-registered with subscription-manager and the auth method is not set to basic auth.

Card ID: CCT-265

This PR is not ready for review! It's a work in progress.
A bunch of tests should be failing right now.

This patch updates the messages shown to the user
when the machine is registered/not-registered with
subscription-manager and the auth method is not set
to basic auth.

Card ID: CCT-265

Signed-off-by: Jason Jerome <[email protected]>
@xiangce xiangce added the client These issues represent work to be done by the "client" team. label Sep 25, 2024
@jenkins-qa-bot
Copy link
Collaborator

Can one of the admins verify this patch?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
client These issues represent work to be done by the "client" team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants