Skip to content

add local windows users guide link to enrollment#34010

Closed
stevenGravy wants to merge 15 commits intomasterfrom
stevenGravy/windowslocalenroll
Closed

add local windows users guide link to enrollment#34010
stevenGravy wants to merge 15 commits intomasterfrom
stevenGravy/windowslocalenroll

Conversation

@stevenGravy
Copy link
Copy Markdown
Contributor

@stevenGravy stevenGravy commented Oct 28, 2023

Includes link to local windows users passwordless setup in discover

image

changelog: Include local windows users guide in discovery

@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.

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.

I don't think "local windows" is the right term for this.

@stevenGravy
Copy link
Copy Markdown
Contributor Author

I don't think "local windows" is the right term for this.

I was also thinking

  • Local Windows Users
  • Local
  • Standalone Windows
  • Non-AD

@stevenGravy stevenGravy changed the title add local windows guide link to enrollment add local windows users guide link to enrollment Nov 3, 2023
@zmb3
Copy link
Copy Markdown
Collaborator

zmb3 commented Nov 14, 2023

@ibeckermayer this is a Q4 goal that's tentatively assigned to you. Up to you whether you want to pick it up from here or start from scratch.

- Added an admonition in `getting-started.mdx` for guiding users about custom Teleport DLL for Windows desktops and integrating with Active Directory.
- Refactored the `WINDOWS_DESKTOPS` array in `resources.tsx` to merge 'Active Directory' and 'Local Windows Users' into a singular 'Windows Desktop' type.
- Updated the pretitle for Windows Desktops in the UI from 'Windows Desktop' to 'Microsoft' for a broader categorization.
@ibeckermayer
Copy link
Copy Markdown
Contributor

IIRC the decision was that we would get rid of the guided AD path entirely. To that end, I've changed it to

image

and added this admonition in the docs to point AD users in the right direction.

@github-actions
Copy link
Copy Markdown
Contributor

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

@github-actions
Copy link
Copy Markdown
Contributor

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

@github-actions
Copy link
Copy Markdown
Contributor

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

@ibeckermayer ibeckermayer added no-changelog Indicates that a PR does not require a changelog entry backport/branch/v12 labels Nov 21, 2023
@ibeckermayer
Copy link
Copy Markdown
Contributor

The goal is actually to remove the AD scripts entirely, so I've updated the link in the admonition to point to active-directory-manual.mdx and created a follow on PR which finishes the job: #34816

@ibeckermayer
Copy link
Copy Markdown
Contributor

This needs to link to a non-AD discover flow. Converting this to a draft for now.

@ibeckermayer ibeckermayer marked this pull request as draft November 22, 2023 00:18
auto-merge was automatically disabled November 22, 2023 00:18

Pull request was converted to draft

@ibeckermayer
Copy link
Copy Markdown
Contributor

Closed in favor of #35231

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog Indicates that a PR does not require a changelog entry size/sm ui

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants