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

[v8] Ensure tctl commands include login instructions #13737

Merged
merged 1 commit into from
Jul 5, 2022

Conversation

ptgott
Copy link
Contributor

@ptgott ptgott commented Jun 22, 2022

Backports #12944

  • Ensure tctl commands include login instructions

See #10192

Add the tctl.mdx partial or a "tsh login" command in some pages that
include example tctl commands. Note that this change does not address
SSO guides, which will be handled separately.

Where a guide requires a complete restructuring to provide full context,
"docs/pages/application-access/guides/connecting-apps.mdx", I've added
"tsh login" instruction above the first tctl command.

  • Respond to PR feedback

@ptgott ptgott changed the title Ensure tctl commands include login instructions [v8] Ensure tctl commands include login instructions Jun 22, 2022
@ptgott ptgott enabled auto-merge (squash) July 5, 2022 21:50
Backports #12944

* Ensure tctl commands include login instructions

See #10192

Add the tctl.mdx partial or a "tsh login" command in some pages that
include example tctl commands. Note that this change does not address
SSO guides, which will be handled separately.

Where a guide requires a complete restructuring to provide full context,
"docs/pages/application-access/guides/connecting-apps.mdx", I've added
"tsh login" instruction above the first tctl command.

* Respond to PR feedback
@ptgott ptgott force-pushed the paul.gottschling/12944-bp-v8 branch from 9020663 to 66e7fb8 Compare July 5, 2022 21:50
@ptgott ptgott merged commit fa526f8 into branch/v8 Jul 5, 2022
@ptgott ptgott deleted the paul.gottschling/12944-bp-v8 branch February 16, 2023 19:02
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