Skip to content

Fixed incorrect component name in install-asgardeo-sdk guide#6224

Open
ChanukaKaweeshwara wants to merge 1 commit into
wso2:masterfrom
ChanukaKaweeshwara:fix-auth-provider-component-name-react-guide
Open

Fixed incorrect component name in install-asgardeo-sdk guide#6224
ChanukaKaweeshwara wants to merge 1 commit into
wso2:masterfrom
ChanukaKaweeshwara:fix-auth-provider-component-name-react-guide

Conversation

@ChanukaKaweeshwara

Copy link
Copy Markdown

I have replaced the incorrect reference with the corrrect in the prose description. The code block correctly mentions AsgardeoProvider from the new asgadeo/rect SDK. but the explanation text still referred to AuthProvider.

Purpose

Related PRs

Test environment

Security checks

I have replaced the incorrect <AuthProvider /> reference with the corrrect <AsgardeoProvider/> in the prose description. The code block correctly mentions AsgardeoProvider from the new asgadeo/rect SDK. but the explanation text still referred to AuthProvider.
@CLAassistant

CLAassistant commented Jun 30, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@coderabbitai

coderabbitai Bot commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

Two lines in en/includes/complete-guides/react/install-asgardeo-sdk.md are updated to replace references to <AuthProvider /> with <AsgardeoProvider />, covering the root-level provider description and the "Easy Access to Authentication" bullet.

React SDK Install Guide

Layer / File(s) Summary
<AuthProvider /><AsgardeoProvider /> rename
en/includes/complete-guides/react/install-asgardeo-sdk.md
Root-level provider description (line 55) and the "Easy Access to Authentication" bullet (line 76) are updated to reference <AsgardeoProvider /> instead of <AuthProvider />.

Suggested labels

Team/Authentication & registration

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The template headings are present, but Purpose, Related PRs, Test environment, and Security checks are left unfilled or unchecked. Add a concrete Purpose, list any related PRs, document the test environment, and complete the security checklist items.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: correcting the component name in the install-asgardeo-sdk guide.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
✨ 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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
en/includes/complete-guides/react/install-asgardeo-sdk.md (1)

55-55: 📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low value

Hyphenate "single-page application."

The static analysis hint flags "single page application" as needing a hyphen. When used as a compound modifier before a noun ("application"), it should be "single-page application."

📝 Proposed fix
-It takes the configuration object with the following values for the single page application defined in the {{product_name}} console.
+It takes the configuration object with the following values for the single-page application defined in the {{product_name}} console.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@en/includes/complete-guides/react/install-asgardeo-sdk.md` at line 55, Update
the prose in the Asgardeo SDK install guide so the phrase in the paragraph using
AsgardeoProvider and the configuration object reads “single-page application”
instead of “single page application.” Keep the rest of the sentence unchanged
and make the same hyphenation wherever this compound modifier appears in the
affected copy.

Source: Linters/SAST tools

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Nitpick comments:
In `@en/includes/complete-guides/react/install-asgardeo-sdk.md`:
- Line 55: Update the prose in the Asgardeo SDK install guide so the phrase in
the paragraph using AsgardeoProvider and the configuration object reads
“single-page application” instead of “single page application.” Keep the rest of
the sentence unchanged and make the same hyphenation wherever this compound
modifier appears in the affected copy.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: b761d0fc-75cd-43fa-b27b-e062648ed17f

📥 Commits

Reviewing files that changed from the base of the PR and between 28901d5 and aa427c4.

📒 Files selected for processing (1)
  • en/includes/complete-guides/react/install-asgardeo-sdk.md

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