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

Update 'Auth indicators' doc string #6716

Closed

Conversation

carma12
Copy link
Contributor

@carma12 carma12 commented Feb 27, 2023

The doc string located in the 'Authentication indicators' ('Services' settings page) was missing the usage explanation for the 'ipd' checkbox option.

Fixes: https://pagure.io/freeipa/issue/9338
Signed-off-by: Carla Martinez carlmart@redhat.com

@carma12 carma12 added needs review Pull Request is waiting for a review and removed needs review Pull Request is waiting for a review labels Feb 27, 2023
The doc string located in the 'Authentication
indicators' ('Services' settings page) was
missing the usage explanation for the 'ipd'
checkbox option.

Fixes: https://pagure.io/freeipa/issue/9338
Signed-off-by: Carla Martinez <carlmart@redhat.com>
@carma12 carma12 force-pushed the update-doc-string-auth-indicators branch from 7bec9e3 to d0c468c Compare February 27, 2023 12:50
@abbra
Copy link
Contributor

abbra commented Feb 27, 2023

LGTM. Please remove the temp commit.

@abbra abbra added ipa-4-9 Mark for backport to ipa 4.9 ipa-4-10 Mark for backport to ipa 4.10 and removed needs review Pull Request is waiting for a review labels Feb 27, 2023
@carma12 carma12 force-pushed the update-doc-string-auth-indicators branch from d0c468c to 6e6c2f3 Compare February 28, 2023 07:56
@flo-renaud flo-renaud added the ack Pull Request approved, can be merged label Mar 2, 2023
@flo-renaud
Copy link
Contributor

Waiting for CI to complete but the PR is ready to be merged otherwise.

@rcritten rcritten added the pushed Pull Request has already been pushed label Mar 2, 2023
@rcritten
Copy link
Contributor

rcritten commented Mar 2, 2023

master:

  • 16b3977 Update 'Auth indicators' doc string

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged ipa-4-9 Mark for backport to ipa 4.9 ipa-4-10 Mark for backport to ipa 4.10 pushed Pull Request has already been pushed
Projects
None yet
4 participants