Skip to content

docs: oauth2-provider fixes #19170

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

Merged
merged 2 commits into from
Aug 4, 2025
Merged

Conversation

hugodutka
Copy link
Contributor

@hugodutka hugodutka commented Aug 4, 2025

Adds the oauth2-provider doc page to the manifest so it's rendered in the docs, fixes formatting in the oauth2-provider doc, and links to it from the MCP doc.

To see the formatting issues, visit https://coder.com/docs/@4bcf44a/admin/integrations/oauth2-provider. To see the doc after the fixes, visit https://coder.com/docs/@f05969a/admin/integrations/oauth2-provider.

@hugodutka hugodutka force-pushed the hugodutka-oauth2-formatting-fixes branch from 3633106 to f05969a Compare August 4, 2025 19:12
@hugodutka hugodutka requested review from ThomasK33 and stirby August 4, 2025 19:13
@@ -718,6 +718,11 @@
"title": "Hashicorp Vault",
"description": "Integrate Coder with Hashicorp Vault",
"path": "./admin/integrations/vault.md"
},
Copy link
Collaborator

Choose a reason for hiding this comment

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

I am fine with this location, but we should probably link to this doc if users find themselves in "External Authentication" looking for Coder as an OAuth2 provider.

Not blocking merge, for future consideration.

@hugodutka hugodutka merged commit c94333d into main Aug 4, 2025
32 checks passed
@hugodutka hugodutka deleted the hugodutka-oauth2-formatting-fixes branch August 4, 2025 19:20
@github-actions github-actions bot locked and limited conversation to collaborators Aug 4, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants