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

[docs] GitHub proxy admin guide and tsh reference #50772

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

greedy52
Copy link
Contributor

@greedy52 greedy52 commented Jan 6, 2025

Copy link

github-actions bot commented Jan 6, 2025

Amplify deployment status

Branch Commit Job ID Status Preview Updated (UTC)
STeve/48762_docs dfdce24 6 ✅SUCCEED steve-48762-docs 2025-01-07 19:57:40

@greedy52 greedy52 added the no-changelog Indicates that a PR does not require a changelog entry label Jan 6, 2025
@greedy52 greedy52 requested a review from ptgott January 7, 2025 14:24
@greedy52 greedy52 marked this pull request as ready for review January 7, 2025 14:24
@@ -0,0 +1,148 @@
---
title: GitHub Integration
Copy link
Contributor

Choose a reason for hiding this comment

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

I would make the title more descriptive about what the integration entails, e.g., "Manage GitHub Access with Teleport" (depending on what's accurate here).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

not sure what's a good name here. i've changed it to Git Access with GitHub as we don't manage access yet. we only proxying git commands. the manage access part will come at a later time for another feature.

docs/pages/includes/edition-prereqs-tabs-enterprise.mdx Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation no-changelog Indicates that a PR does not require a changelog entry size/md
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants