Update broken WS-Trust connector store link - #6282
Conversation
📝 WalkthroughWalkthroughChangesWS-Trust documentation
Suggested labels: Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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
🤖 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/identity-server/5.11.0/docs/learn/configuring-ws-trust-security-token-service.md`:
- Line 12: Update the visible product name to “WSO2 Connector Store” in the
download links at
en/identity-server/5.11.0/docs/learn/configuring-ws-trust-security-token-service.md:12-12,
en/identity-server/6.0.0/docs/guides/identity-federation/configure-ws-trust.md:12-12,
and
en/identity-server/6.1.0/docs/guides/identity-federation/configure-ws-trust.md:12-12;
leave the existing URLs unchanged.
🪄 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: 4ba9fa52-b907-4c09-b99e-2e28b206e46d
📒 Files selected for processing (3)
en/identity-server/5.11.0/docs/learn/configuring-ws-trust-security-token-service.mden/identity-server/6.0.0/docs/guides/identity-federation/configure-ws-trust.mden/identity-server/6.1.0/docs/guides/identity-federation/configure-ws-trust.md
| To download and install the WS-Trust connector: | ||
|
|
||
| 1. Download the [WS-Trust Authenticator](https://store.wso2.com/store/assets/isconnector/details/417e7ef2-76fb-424f-92b3-d5eb58e2efe6) from the WSO2 connector store. | ||
| 1. Download the [WS-Trust Authenticator](https://store.wso2.com/connector/identity-inbound-auth-sts) from the WSO2 connector store. |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Use the official product name capitalization.
The three guides consistently use the updated link, but the visible product name should be “WSO2 Connector Store,” not “WSO2 connector store.”
en/identity-server/5.11.0/docs/learn/configuring-ws-trust-security-token-service.md#L12-L12: capitalize “Connector Store.”en/identity-server/6.0.0/docs/guides/identity-federation/configure-ws-trust.md#L12-L12: capitalize “Connector Store.”en/identity-server/6.1.0/docs/guides/identity-federation/configure-ws-trust.md#L12-L12: capitalize “Connector Store.”
As per coding guidelines, use official product and feature names exactly as defined.
📍 Affects 3 files
en/identity-server/5.11.0/docs/learn/configuring-ws-trust-security-token-service.md#L12-L12(this comment)en/identity-server/6.0.0/docs/guides/identity-federation/configure-ws-trust.md#L12-L12en/identity-server/6.1.0/docs/guides/identity-federation/configure-ws-trust.md#L12-L12
🤖 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/identity-server/5.11.0/docs/learn/configuring-ws-trust-security-token-service.md`
at line 12, Update the visible product name to “WSO2 Connector Store” in the
download links at
en/identity-server/5.11.0/docs/learn/configuring-ws-trust-security-token-service.md:12-12,
en/identity-server/6.0.0/docs/guides/identity-federation/configure-ws-trust.md:12-12,
and
en/identity-server/6.1.0/docs/guides/identity-federation/configure-ws-trust.md:12-12;
leave the existing URLs unchanged.
Source: Coding guidelines
|
I Reopened the pull request with updated store link |
|
@mathup15 thank you for the contribution. Could you please address the review comments so that we can merge the PR? |
Purpose
Updated the broken WSO2 Store connector links in the WS-Trust documentation for WSO2 Identity Server 5.11.0, 6.0.0, and 6.1.0.
Resolves #6259
Related PRs
N/A
Test environment
Security checks