Skip to content

Update auto discovery doc to include self-hosted web UI flow#36002

Closed
kimlisa wants to merge 1 commit intomasterfrom
lisa/update-discovery-doc
Closed

Update auto discovery doc to include self-hosted web UI flow#36002
kimlisa wants to merge 1 commit intomasterfrom
lisa/update-discovery-doc

Conversation

@kimlisa
Copy link
Copy Markdown
Contributor

@kimlisa kimlisa commented Dec 22, 2023

Adds little notes to the aws auto discovery doc if user came from the web UI: #36027

changed in: https://docs-2dew2l61w-goteleport.vercel.app/docs/database-access/guides/aws-discovery/

screenshot:
image

image

@kimlisa kimlisa changed the title wip Update auto discovery doc to include self-hosted web UI flow Dec 22, 2023
@kimlisa kimlisa force-pushed the lisa/update-discovery-doc branch from 3333d0c to ac90e18 Compare December 22, 2023 06:36
@kimlisa kimlisa force-pushed the lisa/update-discovery-doc branch from ac90e18 to c1f72b6 Compare December 22, 2023 23:11
@kimlisa kimlisa force-pushed the lisa/update-discovery-doc branch from c1f72b6 to d1ce272 Compare December 22, 2023 23:11
@gravitational gravitational deleted a comment from github-actions Bot Dec 22, 2023
@gravitational gravitational deleted a comment from github-actions Bot Dec 22, 2023
@kimlisa kimlisa force-pushed the lisa/update-discovery-doc branch from d1ce272 to 4f1d5c6 Compare December 22, 2023 23:19
@gravitational gravitational deleted a comment from github-actions Bot Dec 22, 2023
@github-actions
Copy link
Copy Markdown
Contributor

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

@gravitational gravitational deleted a comment from github-actions Bot Dec 22, 2023
@kimlisa kimlisa force-pushed the lisa/update-discovery-doc branch from 4f1d5c6 to c91f6ee Compare December 22, 2023 23:29
@kimlisa kimlisa marked this pull request as ready for review December 22, 2023 23:29
@kimlisa kimlisa added no-changelog Indicates that a PR does not require a changelog entry backport/branch/v14 labels Dec 22, 2023
@github-actions
Copy link
Copy Markdown
Contributor

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

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.

Which parts of this docs page do Web UI users need to follow? Can we incorporate those instructions into the Web UI instead? Otherwise, we get the worst of both worlds:

  • Web UI users need to refer to the docs site, which can be awkward and requires context switching.
  • Docs site users need to filter out the noise created by instructions for Web UI users, which makes the docs site less pleasant to use.

@kimlisa
Copy link
Copy Markdown
Contributor Author

kimlisa commented Jan 4, 2024

Which parts of this docs page do Web UI users need to follow? Can we incorporate those instructions into the Web UI instead? Otherwise, we get the worst of both worlds:

  • Web UI users need to refer to the docs site, which can be awkward and requires context switching.
  • Docs site users need to filter out the noise created by instructions for Web UI users, which makes the docs site less pleasant to use.

you're right, on second thought it isn't that much doc to put in the web UI (or even better marco is currently working on a script to replace docs)

@kimlisa kimlisa closed this Jan 4, 2024
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/sm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants