Conversation
Added links for accessing Asgardeo Console in US and EU regions.
|
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthroughThe create-asgardeo-account.md documentation was modified to replace a single console link with region-specific labeled sections and separate curl examples for Asgardeo - US and Asgardeo - EU (lines changed: +10/-2). Changes
Suggested labels
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Warning Review ran into problems🔥 ProblemsTimed out fetching pipeline failures after 30000ms 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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
en/asgardeo/docs/get-started/create-asgardeo-account.md (1)
21-22:⚠️ Potential issue | 🟠 MajorResolve the markdown lint failure (MD012) by removing the extra blank line.
The pipeline reports consecutive blank lines at Line 21; this blocks markdown lint.
Suggested edit
- - !!! tip "Access the Asgardeo Console"🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@en/asgardeo/docs/get-started/create-asgardeo-account.md` around lines 21 - 22, Remove the extra blank line immediately after the admonition opener '!!! tip "Access the Asgardeo Console"' so there are no consecutive blank lines; edit the markdown around that admonition to ensure only a single newline follows the '!!! tip' line, saving the file to resolve the MD012 lint error.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@en/asgardeo/docs/get-started/create-asgardeo-account.md`:
- Around line 24-38: Update the paragraph that starts "As an organization owner,
or an administrator, you can manage all organizations, applications and users
that you create using the Asgardeo Console" to use singular/plural and plain
language (e.g., "As an organization owner or administrator, you can manage the
organizations, applications, and users you create from the Asgardeo Console")
and replace the raw URL code blocks for "https://console.asgardeo.io/" and
"https://console.eu.asgardeo.io/" with a short bullet list using descriptive
link text such as "- [Asgardeo Console (US)](https://console.asgardeo.io/)" and
"- [Asgardeo Console (EU)](https://console.eu.asgardeo.io/)". Ensure the
sentence no longer says "following link" but "following links" and remove the
```curl``` fences so the URLs are presented as regular Markdown links.
---
Outside diff comments:
In `@en/asgardeo/docs/get-started/create-asgardeo-account.md`:
- Around line 21-22: Remove the extra blank line immediately after the
admonition opener '!!! tip "Access the Asgardeo Console"' so there are no
consecutive blank lines; edit the markdown around that admonition to ensure only
a single newline follows the '!!! tip' line, saving the file to resolve the
MD012 lint error.
🪄 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: 80a3ece4-b16c-4ca2-845b-a436479678b5
📒 Files selected for processing (1)
en/asgardeo/docs/get-started/create-asgardeo-account.md
| As an organization owner, or an administrator, you can manage all organizations, applications and users that you create using the Asgardeo Console. You can access the console using the following link. | ||
|
|
||
|
|
||
| Asgardeo - US | ||
| ```curl | ||
|
|
||
| https://console.asgardeo.io/ | ||
|
|
||
| ``` | ||
|
|
||
| Asgardeo - EU | ||
| ```curl | ||
|
|
||
| https://console.eu.asgardeo.io/ | ||
|
|
||
| ``` |
There was a problem hiding this comment.
Fix singular wording and replace raw URL code blocks with descriptive links.
The sentence says “following link” but now includes two endpoints. Also, using curl fences for plain URLs is misleading. Prefer a short list with descriptive link text.
Suggested edit
- As an organization owner, or an administrator, you can manage all organizations, applications and users that you create using the Asgardeo Console. You can access the console using the following link.
-
- Asgardeo - US
- ```curl
-
- https://console.asgardeo.io/
-
- ```
-
- Asgardeo - EU
- ```curl
-
- https://console.eu.asgardeo.io/
-
- ```
+ As an organization owner or administrator, you can manage the organizations, applications, and users that you create from the Asgardeo Console. You can access the console using the following links:
+
+ - [Asgardeo Console (US)](https://console.asgardeo.io/)
+ - [Asgardeo Console (EU)](https://console.eu.asgardeo.io/)As per coding guidelines, “Use descriptive link text and do not paste raw URLs as link text” and “Use plain language and short sentences.”
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@en/asgardeo/docs/get-started/create-asgardeo-account.md` around lines 24 -
38, Update the paragraph that starts "As an organization owner, or an
administrator, you can manage all organizations, applications and users that you
create using the Asgardeo Console" to use singular/plural and plain language
(e.g., "As an organization owner or administrator, you can manage the
organizations, applications, and users you create from the Asgardeo Console")
and replace the raw URL code blocks for "https://console.asgardeo.io/" and
"https://console.eu.asgardeo.io/" with a short bullet list using descriptive
link text such as "- [Asgardeo Console (US)](https://console.asgardeo.io/)" and
"- [Asgardeo Console (EU)](https://console.eu.asgardeo.io/)". Ensure the
sentence no longer says "following link" but "following links" and remove the
```curl``` fences so the URLs are presented as regular Markdown links.
Updated the documentation to clarify access links for the Asgardeo Console.
| As an organization owner, or an administrator, you can manage all organizations, applications and users that you create using the Asgardeo Console. You can access the console using the following links. | ||
|
|
||
| Asgardeo - US | ||
| ```curl | ||
|
|
||
| https://console.asgardeo.io/ | ||
|
|
||
| ``` | ||
|
|
||
| Asgardeo - EU | ||
| ```curl | ||
|
|
||
| https://console.eu.asgardeo.io/ | ||
|
|
||
| ``` |
There was a problem hiding this comment.
WDYT about this (ai generated) version? it seems more polished than the current version IMO. URLs are not bash commands to wrap in backticks
| As an organization owner, or an administrator, you can manage all organizations, applications and users that you create using the Asgardeo Console. You can access the console using the following links. | |
| Asgardeo - US | |
| ```curl | |
| https://console.asgardeo.io/ | |
| ``` | |
| Asgardeo - EU | |
| ```curl | |
| https://console.eu.asgardeo.io/ | |
| ``` | |
| As an organization owner or administrator, you can manage your organizations, applications, and users through the Asgardeo Console. Make sure to use the appropriate console URL based on the chosen region during registration. | |
| - US Region: https://console.asgardeo.io/ | |
| - EU Region: https://console.eu.asgardeo.io/ |
Added link for accessing Asgardeo Console in EU region.
Purpose
Providing a reference on the docs for users to correctly access the 2 Asgardeo deployments.