Skip to content

Add an overview of the Teleport documentation#34476

Merged
ptgott merged 2 commits intomasterfrom
paul.gottschling/34038-ov
Jan 25, 2024
Merged

Add an overview of the Teleport documentation#34476
ptgott merged 2 commits intomasterfrom
paul.gottschling/34038-ov

Conversation

@ptgott
Copy link
Copy Markdown
Contributor

@ptgott ptgott commented Nov 10, 2023

Closes #34038

Help visitors to the Teleport documentation find their way through the docs site by including a description of each docs page in an overview guide.

This change is based on adapting the former Teleport index page (prior to #27703).

@ptgott
Copy link
Copy Markdown
Contributor Author

ptgott commented Nov 10, 2023

Per a request from @stevenGravy

@github-actions
Copy link
Copy Markdown
Contributor

The PR changelog entry failed validation: Changelog entry not found in the PR body. Please add a "no-changelog" label to the PR, or changelog lines starting with changelog: followed by the changelog entries for the PR.

@github-actions
Copy link
Copy Markdown
Contributor

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

@stevenGravy stevenGravy added the no-changelog Indicates that a PR does not require a changelog entry label Nov 10, 2023
@ptgott ptgott force-pushed the paul.gottschling/34038-ov branch from 921c4d0 to 276fee0 Compare November 13, 2023 22:54
@github-actions
Copy link
Copy Markdown
Contributor

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

@zmb3
Copy link
Copy Markdown
Collaborator

zmb3 commented Nov 14, 2023

Are we still moving to a new docs engine this quarter? If so, will this go out of date as soon as we do that?

@ptgott
Copy link
Copy Markdown
Contributor Author

ptgott commented Nov 14, 2023

Are we still moving to a new docs engine this quarter? If so, will this go out of date as soon as we do that?

We don't have any plans to change the structure of the docs/pages directory tree alongside the migration, so the only thing that might change is how we organize the versions of the docs site. So far, it looks like we'll still use a dropdown menu for switching between versions. Otherwise, we could change the second paragraph.

@stevenGravy Since you made the request, I'd be interested to know how urgent this is—should we wait until the end of the quarter to check whether the migration will require us to fundamentally edit this page, or would it be valuable to have this overview for the next couple of months?

@github-actions
Copy link
Copy Markdown
Contributor

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

@stevenGravy
Copy link
Copy Markdown
Contributor

Are we still moving to a new docs engine this quarter? If so, will this go out of date as soon as we do that?

We don't have any plans to change the structure of the docs/pages directory tree alongside the migration, so the only thing that might change is how we organize the versions of the docs site. So far, it looks like we'll still use a dropdown menu for switching between versions. Otherwise, we could change the second paragraph.

@stevenGravy Since you made the request, I'd be interested to know how urgent this is—should we wait until the end of the quarter to check whether the migration will require us to fundamentally edit this page, or would it be valuable to have this overview for the next couple of months?

Yes, this will still be valuable. We often have new team operator members joining existing Teleport deployments and this helps to orient them on what the documentation provides.

@github-actions
Copy link
Copy Markdown
Contributor

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

@ptgott
Copy link
Copy Markdown
Contributor Author

ptgott commented Nov 27, 2023

@stevenGravy Just following up to see if you have time to review this. Thanks!

@github-actions
Copy link
Copy Markdown
Contributor

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

@github-actions
Copy link
Copy Markdown
Contributor

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

@github-actions
Copy link
Copy Markdown
Contributor

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

@ptgott ptgott force-pushed the paul.gottschling/34038-ov branch from 1898af5 to 6674b11 Compare January 2, 2024 18:58
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jan 2, 2024

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

@github-actions
Copy link
Copy Markdown
Contributor

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

Comment thread docs/pages/documentation-overview.mdx Outdated
Comment thread docs/pages/documentation-overview.mdx Outdated
Comment thread docs/config.json Outdated
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Should this be the default page that we show at goteleport.com/docs?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

The overview is based on what used to be the default page. We changed the landing page to the previous "Install Teleport on a Linux Server" guide after a recommendation from Ev.

Comment thread docs/pages/documentation-overview.mdx Outdated
Comment thread docs/pages/documentation-overview.mdx Outdated
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
Teleport is protocol aware and provides functionality that is unique to each
Teleport is protocol-aware and provides functionality that is unique to each

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It's pretty common in style guides to allow for excluding hyphens in compound modifiers that take place after nouns, e.g., https://www.grammarly.com/blog/hyphen/

Closes #34038

Help visitors to the Teleport documentation find their way through the
docs site by including a description of each docs page in an overview
guide.

This change is based on adapting the former Teleport index page (prior
to #27703).
@ptgott
Copy link
Copy Markdown
Contributor Author

ptgott commented Jan 24, 2024

Going to hold off on merging for a bit until I address this comment.

@github-actions
Copy link
Copy Markdown
Contributor

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

@ptgott ptgott enabled auto-merge January 25, 2024 18:36
@ptgott ptgott added this pull request to the merge queue Jan 25, 2024
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jan 25, 2024
@ptgott ptgott added this pull request to the merge queue Jan 25, 2024
Merged via the queue into master with commit d217b87 Jan 25, 2024
@ptgott ptgott deleted the paul.gottschling/34038-ov branch January 25, 2024 19:38
@public-teleport-github-review-bot
Copy link
Copy Markdown

@ptgott See the table below for backport results.

Branch Result
branch/v14 Create PR
branch/v15 Create PR

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.

Add an overview of the docs site

3 participants