Skip to content

docs: make consistent access request plugins helm configuration and instructions#24654

Merged
stevenGravy merged 5 commits intomasterfrom
stevenGravy/docs/helmconfig
Apr 18, 2023
Merged

docs: make consistent access request plugins helm configuration and instructions#24654
stevenGravy merged 5 commits intomasterfrom
stevenGravy/docs/helmconfig

Conversation

@stevenGravy
Copy link
Copy Markdown
Contributor

  • use proxy address as address example in config matching doc instrs
  • use same language for cloud and self-hosted address as executable instructions.

Copy link
Copy Markdown
Collaborator

@zmb3 zmb3 left a comment

Choose a reason for hiding this comment

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

The change looks fine, though the commit message is pretty hard to understand.

Comment thread docs/pages/includes/plugins/config-toml-teleport.mdx Outdated
Comment thread examples/resources/plugins/teleport-discord-helm.yaml Outdated
@ptgott ptgott disabled auto-merge April 18, 2023 18:58
Copy link
Copy Markdown
Contributor

@ptgott ptgott left a comment

Choose a reason for hiding this comment

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

Approved with one suggestion to address my feedback from earlier

Comment thread examples/resources/plugins/teleport-slack-helm.yaml Outdated
@stevenGravy stevenGravy enabled auto-merge April 18, 2023 19:15
@stevenGravy stevenGravy added this pull request to the merge queue Apr 18, 2023
Merged via the queue into master with commit bebef91 Apr 18, 2023
@stevenGravy stevenGravy deleted the stevenGravy/docs/helmconfig branch April 18, 2023 19:44
@public-teleport-github-review-bot
Copy link
Copy Markdown

@stevenGravy See the table below for backport results.

Branch Result
branch/v10 Failed
branch/v11 Create PR
branch/v12 Create PR
branch/v13 Create PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants