Skip to content

Fix documentation issues in OAuth2 and Public Clients - #6318

Open
sanu-dinee wants to merge 4 commits into
wso2:masterfrom
sanu-dinee:master
Open

Fix documentation issues in OAuth2 and Public Clients#6318
sanu-dinee wants to merge 4 commits into
wso2:masterfrom
sanu-dinee:master

Conversation

@sanu-dinee

@sanu-dinee sanu-dinee commented Aug 25, 2026

Copy link
Copy Markdown

Purpose

This PR fixes two documentation issues in the OAuth2 and Public Clients documentation:

  • Fixes invalid JSON in the OAuth2 grant type example by adding the missing comma after the scope property.
  • Removes a duplicate sentence from the Public Clients documentation.

These are documentation-only changes.

Related PRs

None.

Test environment

Documentation-only changes. No code changes were made, so no runtime testing is required.

Security checks

  • No security-sensitive code was changed.
  • No credentials, secrets, or configuration changes were introduced.

@CLAassistant

CLAassistant commented Aug 25, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@coderabbitai

coderabbitai Bot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro Plus

Run ID: f7f71467-a67d-410b-ad10-9fe02a1f4dd3

📥 Commits

Reviewing files that changed from the base of the PR and between 5876fa8 and 10335ac.

📒 Files selected for processing (2)
  • en/includes/complete-guides/fesecurity/public-clients.md
  • en/includes/references/grant-types.md

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

Changes

Documentation examples

Layer / File(s) Summary
Correct documentation examples
en/includes/complete-guides/fesecurity/public-clients.md, en/includes/references/grant-types.md
The public client response example now has a closing code fence and no longer includes the removed sentence. The authorization code grant response example adds a trailing comma after scope.

Suggested labels: Team/API Access Mgt & Authorization

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly summarizes the two documentation fixes in OAuth2 and Public Clients documentation.
Description check ✅ Passed The description includes all required sections and explains the documentation fixes, related PRs, test environment, and security considerations. It is sufficient for documentation-only changes.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (2 skipped: 2 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants