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

Fix roleToRecipients descriptions #46737

Merged
merged 1 commit into from
Oct 7, 2024
Merged

Conversation

ptgott
Copy link
Contributor

@ptgott ptgott commented Sep 18, 2024

Closes #45287

Access Request plugin guides describe the value of the roleToRecipients field in Access Request plugin Helm charts as taking either a single string or an array of strings as the value of each mapping. This change fixes the error in all Access Request plugin guides that include this Helm chart value.

Copy link

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

Copy link

github-actions bot commented Oct 1, 2024

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

Closes #45287

Access Request plugin guides describe the value of the
`roleToRecipients` field in Access Request plugin Helm charts as taking
either a single string or an array of strings as the value of each
mapping. This change fixes the error in all Access Request plugin guides
that include this Helm chart value.
@ptgott
Copy link
Contributor Author

ptgott commented Oct 7, 2024

@mmcallister Checking to see if you have time to review this one. Thanks!

Copy link

github-actions bot commented Oct 7, 2024

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

@ptgott ptgott added this pull request to the merge queue Oct 7, 2024
Merged via the queue into master with commit 7f41f1c Oct 7, 2024
39 of 40 checks passed
@ptgott ptgott deleted the paul.gottschling/45287-access branch October 7, 2024 18:56
@public-teleport-github-review-bot

@ptgott See the table below for backport results.

Branch Result
branch/v14 Create PR
branch/v15 Create PR
branch/v16 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.

RoleToRecipients Helm Chart value does not accept string value
3 participants