Release: Self managed oauth clients#31213
Conversation
Review⏸️ Automatic reviews for this PR are paused. This PR has already received 2 automatic reviews. To run another review, a codeowner can comment
Warnings (2)
CommandsOnly codeowners can run commands. Post a comment with the command to trigger it.
|
Co-authored-by: Adam Bouhmad <adbouhmad@gmail.com>
Co-authored-by: Adam Bouhmad <adbouhmad@gmail.com>
Co-authored-by: Adam Bouhmad <adbouhmad@gmail.com>
|
/bonk review |
There was a problem hiding this comment.
Thanks for the PR! The new OAuth documentation is well-structured overall. I found a few issues that need addressing before this is ready to merge:
- FeatureTable ID mismatch in
index.mdx— it references the SSO plan entry instead of the new self-managed OAuth entry. - Bare URL in the
self-managed-oauth.mdxpartial — won't render as a clickable link. - Button destination inconsistency in
authorizing-an-application.mdx— the text refers to a profile page but the button links to the OAuth clients management page. - Inconsistent permission name in
create-an-oauth-client.mdx— singular vs plural form.
Co-authored-by: irvinebroque <irvinebroque@users.noreply.github.com>
|
I've completed the review on PR #31213 and posted a batch review with four inline comments. Here are the key issues I flagged:
You can view the review here: #31213 (review) |
Co-authored-by: ask-bonk[bot] <249159057+ask-bonk[bot]@users.noreply.github.com>
Co-authored-by: ask-bonk[bot] <249159057+ask-bonk[bot]@users.noreply.github.com>
Summary
Docs release for the launch of public self-managed oauth clients.
Documentation checklist