Skip to content

docs: tsh puttyconfig#29458

Merged
webvictim merged 8 commits intomasterfrom
gus/docs/tsh-putty
Jul 31, 2023
Merged

docs: tsh puttyconfig#29458
webvictim merged 8 commits intomasterfrom
gus/docs/tsh-putty

Conversation

@webvictim
Copy link
Copy Markdown
Contributor

@webvictim webvictim commented Jul 24, 2023

Adds documentation for tsh puttyconfig.

No backport required, as this functionality is only in master and won't be released until Teleport 14 comes out.

@webvictim webvictim changed the title gus/docs/tsh putty docs: tsh puttyconfig Jul 24, 2023
@webvictim webvictim self-assigned this Jul 24, 2023
@webvictim webvictim changed the title docs: tsh puttyconfig docs: tsh puttyconfig Jul 24, 2023
@webvictim webvictim marked this pull request as ready for review July 24, 2023 18:29
@webvictim webvictim requested a review from stevenGravy July 24, 2023 18:29
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.

🚀

Copy link
Copy Markdown
Contributor

@lsgunn-teleport lsgunn-teleport left a comment

Choose a reason for hiding this comment

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

Some suggestions with the biggest change around converting code comments into steps.

Comment thread docs/pages/connect-your-client/putty.mdx Outdated
Comment thread docs/pages/connect-your-client/putty.mdx Outdated
Comment thread docs/pages/connect-your-client/putty.mdx Outdated
Comment thread docs/pages/connect-your-client/putty.mdx Outdated
Comment thread docs/pages/connect-your-client/putty.mdx Outdated
Comment thread docs/pages/connect-your-client/putty.mdx Outdated
Comment thread docs/pages/connect-your-client/putty.mdx Outdated
Comment thread docs/pages/connect-your-client/putty.mdx Outdated
Comment thread docs/pages/connect-your-client/putty.mdx Outdated
Comment thread docs/pages/connect-your-client/putty.mdx Outdated
@webvictim
Copy link
Copy Markdown
Contributor Author

webvictim commented Jul 26, 2023

Thanks for the review @lsgunn-teleport!

@webvictim webvictim enabled auto-merge July 26, 2023 20:08
@webvictim webvictim added this pull request to the merge queue Jul 31, 2023
Comment on lines +37 to +42

```bash
C:\Users\gus>tsh login --proxy=teleport.example.com
```

This command retrieves your user certificates and saves them in a local file in the `%USERPROFILE%/.tsh` directory.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is a really minor thing for future reference. If you indent three spaces under a step, the text lines up nicely.
I'll make a quick pass through to fix this when I have a few minutes, but no reason to hold this PR up for it.

C:\Users\gus>tsh login --proxy=teleport.example.com

@zmb3 zmb3 added release-notes A reminder label to into the release notes of the Teleport Release. changelog labels Jul 31, 2023
Merged via the queue into master with commit e68c2db Jul 31, 2023
@webvictim webvictim deleted the gus/docs/tsh-putty branch July 31, 2023 19:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation release-notes A reminder label to into the release notes of the Teleport Release. size/md

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants