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

Simplify how-to guides #47586

Merged
merged 1 commit into from
Oct 17, 2024
Merged

Simplify how-to guides #47586

merged 1 commit into from
Oct 17, 2024

Conversation

ptgott
Copy link
Contributor

@ptgott ptgott commented Oct 15, 2024

Closes #38931

Remove "Static" and "Dynamic" configuration branching. Use the dynamic approach unless the guide already shows readers how to use the static approach elsewhere, e.g., using teleport db configure create.

For example, in the AWS OpenSearch guide, creating a Teleport configuration is required for the guide anyway, so this change uses only the static approach.

Ignore edge cases where the branching has some qualities that are unique to a specific guide:

  • application-access/cloud-apis/google-cloud.mdx: includes branching on authentication preference type.
  • application-access/guides/connecting-apps.mdx because it explains syntax differences between the static and dynamic approaches.
  • enroll-aws-databases/aws-cross-account.mdx: branches using three tabs, so editing requires another approach than the one this change applies.

This change edits only how-to guides, not conceptual guides, since the goal of a how-to guide is to get the user to an end state as quickly as possible.

Copy link

🤖 Vercel preview here: https://docs-d7va1isjm-goteleport.vercel.app/docs/ver/preview

@ptgott ptgott added the no-changelog Indicates that a PR does not require a changelog entry label Oct 17, 2024
@ptgott ptgott enabled auto-merge October 17, 2024 12:47
@ptgott ptgott force-pushed the paul.gottschling/38931-branching branch from 0bac356 to 4bb9a43 Compare October 17, 2024 12:52
Copy link

🤖 Vercel preview here: https://docs-1qdnpgxbh-goteleport.vercel.app/docs/ver/preview

Closes #38931

Remove "Static" and "Dynamic" configuration branching. Use the dynamic
approach unless the guide already shows readers how to use the static
approach elsewhere, e.g., using `teleport db configure create`.

For example, in the AWS OpenSearch guide, creating a Teleport
configuration is required for the guide anyway, so this change uses only
the static approach.

Ignore edge cases where the branching has some qualities that are unique
to a specific guide:

- `application-access/cloud-apis/google-cloud.mdx`: includes branching
  on authentication preference type.
- `application-access/guides/connecting-apps.mdx` because it explains
  syntax differences between the static and dynamic approaches.
- `enroll-aws-databases/aws-cross-account.mdx`: branches using three
  tabs, so editing requires another approach than the one this change
  applies.

This change edits only how-to guides, not conceptual guides, since the
goal of a how-to guide is to get the user to an end state as quickly as
possible.
Copy link

🤖 Vercel preview here: https://docs-4nugj10kx-goteleport.vercel.app/docs/ver/preview

@ptgott ptgott added this pull request to the merge queue Oct 17, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 17, 2024
@ptgott ptgott added this pull request to the merge queue Oct 17, 2024
Merged via the queue into master with commit fdf9e25 Oct 17, 2024
40 checks passed
@ptgott ptgott deleted the paul.gottschling/38931-branching branch October 17, 2024 14:48
@public-teleport-github-review-bot

@ptgott See the table below for backport results.

Branch Result
branch/v16 Create PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/branch/v16 documentation no-changelog Indicates that a PR does not require a changelog entry size/sm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove static/dynamic config branching in agent how-to guides
3 participants