Skip to content

Add Asgardeo docs for enhanced organization authentication#6134

Open
SujanSanjula96 wants to merge 2 commits into
wso2:masterfrom
SujanSanjula96:enhanced-b2b-login-2
Open

Add Asgardeo docs for enhanced organization authentication#6134
SujanSanjula96 wants to merge 2 commits into
wso2:masterfrom
SujanSanjula96:enhanced-b2b-login-2

Conversation

@SujanSanjula96
Copy link
Copy Markdown
Contributor

Purpose

$Subject

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 11, 2026

Review Change Stack

Warning

Rate limit exceeded

@SujanSanjula96 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 26 minutes and 16 seconds before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: 7a779b4e-1031-4b1e-b735-22340bc6d400

📥 Commits

Reviewing files that changed from the base of the PR and between 0bcbeec and b129e55.

📒 Files selected for processing (1)
  • en/includes/guides/organization-management/organization-discovery/organization-discovery-enhanced-flow.md
📝 Walkthrough

Walkthrough

This pull request introduces product-aware documentation for Asgardeo organization login and discovery workflows. New Asgardeo-specific wrapper pages set template variables and include shared guides. The shared guides now contain conditional Jinja blocks that render different content for Asgardeo versus WSO2 Identity Server based on discovery parameters, prompt labels, and supported feature sets.

Changes

Organization-aware login and discovery documentation

Layer / File(s) Summary
Navigation structure
en/asgardeo/mkdocs.yml
Organization-based login subsection added under Customize organizations, linking to overview, organization discovery, app-native authentication, legacy approach, and migration guides.
Asgardeo documentation wrappers
en/asgardeo/docs/guides/organization-management/organization-discovery/organization-discovery.md, en/asgardeo/docs/guides/organization-management/organization-login/organization-login-overview.md, en/asgardeo/docs/guides/organization-management/organization-login/legacy-approach.md, en/asgardeo/docs/guides/organization-management/organization-login/migrate-to-enhanced-approach.md, en/asgardeo/docs/guides/organization-management/organization-login/organization-login-with-app-native-authentication.md
Asgardeo-specific wrapper pages set host_name and organization_path_param template variables and include corresponding shared guide content.
Organization discovery product conditionals
en/includes/guides/organization-management/organization-discovery/organization-discovery-enhanced-flow.md
Shared organization discovery guide branches on product_name: Asgardeo displays organization name prompt and Name/ID/Email Domain discovery types; other products show organization handle prompt with additional Handle-Based discovery. Configure default parameter section and Organization Handle subsection restricted to non-Asgardeo products.
Organization login discovery parameter guidance
en/includes/guides/organization-management/organization-login/organization-login-overview.md
Shared login guide conditionally documents organization discovery parameters: Asgardeo recommends org, orgId, or email domain hint; other products additionally include orgHandle.

Suggested labels

Team/B2B

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is severely incomplete, containing only '$Subject' as placeholder text instead of the required template sections like Purpose, Related PRs, Test environment, and Security checks. Complete the PR description using the template: provide actual Purpose details, list any Related PRs, specify Test environment details, and check the Security checks checkboxes with explanations.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Add Asgardeo docs for enhanced organization authentication' accurately summarizes the main change—adding documentation for Asgardeo's enhanced organization authentication features.
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.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ 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 and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 2

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

Inline comments:
In
`@en/includes/guides/organization-management/organization-discovery/organization-discovery-enhanced-flow.md`:
- Around line 52-53: Add a blank line between the closing Jinja conditional
block "{% if product_name == "WSO2 Identity Server" %}" and the following
heading "## Configure default parameter for organization discovery" so the
heading is separated by an empty line (fixes MD022); update the template section
containing that Jinja if and the heading to insert one empty line immediately
before the "## Configure default parameter for organization discovery" line.
- Around line 116-119: Thematic break '---' is placed immediately before the
"Directly access the organization" heading causing linter warnings; update the
markdown to ensure a blank line both before and after the horizontal rule and
confirm the heading uses the correct style (e.g., "## Directly access the
organization") so there is at least one empty line separating the preceding
content, the '---' thematic break, and the heading; locate the '---' and the
"Directly access the organization" heading in the snippet and insert the needed
blank line(s) to satisfy the linter.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: ed2d0266-8cfc-4ca9-bb5b-d1fdc30c27be

📥 Commits

Reviewing files that changed from the base of the PR and between 9470df5 and 0bcbeec.

⛔ Files ignored due to path filters (3)
  • en/asgardeo/docs/assets/img/guides/organization/organization-login/discovery-page-org-name.png is excluded by !**/*.png
  • en/asgardeo/docs/assets/img/guides/organization/organization-login/enhanced-org-authentication.png is excluded by !**/*.png
  • en/asgardeo/docs/assets/img/guides/organization/organization-login/sso-option.png is excluded by !**/*.png
📒 Files selected for processing (8)
  • en/asgardeo/docs/guides/organization-management/organization-discovery/organization-discovery.md
  • en/asgardeo/docs/guides/organization-management/organization-login/legacy-approach.md
  • en/asgardeo/docs/guides/organization-management/organization-login/migrate-to-enhanced-approach.md
  • en/asgardeo/docs/guides/organization-management/organization-login/organization-login-overview.md
  • en/asgardeo/docs/guides/organization-management/organization-login/organization-login-with-app-native-authentication.md
  • en/asgardeo/mkdocs.yml
  • en/includes/guides/organization-management/organization-discovery/organization-discovery-enhanced-flow.md
  • en/includes/guides/organization-management/organization-login/organization-login-overview.md

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant