Skip to content

Fix three broken external links in connect-with-mattr tutorial#6026

Open
mathushihanratnatheepan wants to merge 2 commits intowso2:masterfrom
mathushihanratnatheepan:mathushihanratnatheepan--fix-broken-mattr-links
Open

Fix three broken external links in connect-with-mattr tutorial#6026
mathushihanratnatheepan wants to merge 2 commits intowso2:masterfrom
mathushihanratnatheepan:mathushihanratnatheepan--fix-broken-mattr-links

Conversation

@mathushihanratnatheepan
Copy link
Copy Markdown

@mathushihanratnatheepan mathushihanratnatheepan commented Mar 27, 2026

Purpose

Updated three broken external links in the MATTR integration tutorial
that return 404 errors following MATTR's restructuring of their docs
and GitHub repositories.

Affected page: https://is.docs.wso2.com/en/latest/references/tutorials/connect-with-mattr/

Changes made

Link 1 — MATTR-supported credential claim mappings

Link 2 — MATTR mobile wallet app details

Link 3 — MATTR sample application

Related PRs

None

Test environment

N/A - documentation change only

Security checks

  • Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets

Summary by CodeRabbit

  • Documentation
    • Updated external documentation links to MATTR resources: replaced prior claim-mapping reference with MATTR credential configuration guidance, pointed wallet setup instructions to MATTR's getting-started guide, and generalized the sample verification app link to the MATTR sample-apps repository.

Updated three broken external links that return 404 errors due to MATTR restructuring their docs and GitHub repositories.
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Mar 27, 2026

CLA assistant check
All committers have signed the CLA.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 27, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 5e4ec17b-0de9-43ce-a050-f7f93cf4c0ef

📥 Commits

Reviewing files that changed from the base of the PR and between 371a3a4 and c90721b.

📒 Files selected for processing (1)
  • en/identity-server/7.2.0/docs/references/tutorials/connect-with-mattr.md
✅ Files skipped from review due to trivial changes (1)
  • en/identity-server/7.2.0/docs/references/tutorials/connect-with-mattr.md

Walkthrough

Replaced several external links in the MATTR integration tutorial: updated the claimMappings reference to MATTR's credential configurations guide, changed the wallet app instructions link to go-hold/getting-started, broadened the sample-apps GitHub URL, and added a trailing newline.

Changes

Cohort / File(s) Summary
Documentation Links Update
en/identity-server/7.2.0/docs/references/tutorials/connect-with-mattr.md
Replaced claimMappings link with MATTR "credential configurations" guide, updated MATTR wallet app instructions link to go-hold/getting-started, broadened GitHub sample-apps URL from a subpath to the repository root, and added a trailing newline.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 I hopped through links with nimble feet,
Swapped old paths for guides more neat,
Broadened a URL with a twirl,
Placed a newline — a tiny pearl,
Docs refreshed, I give a cheer,
Little changes, wag the ear.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: fixing three broken external links in the connect-with-mattr tutorial documentation.
Description check ✅ Passed The description is well-structured with clear sections covering purpose, affected page, specific changes for each link, related PRs, test environment, and security checks.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ 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: 1

🧹 Nitpick comments (1)
en/identity-server/7.2.0/docs/references/tutorials/connect-with-mattr.md (1)

182-182: Use more descriptive link text for the wallet documentation.

The link text "this documentation" is vague and does not help users understand the destination before clicking. Per the coding guidelines: "Use descriptive link text."

Suggested improvement
-        See [this documentation](https://learn.mattr.global/docs/holding/go-hold/getting-started) for details.
+        See the [GO Hold getting started guide](https://learn.mattr.global/docs/holding/go-hold/getting-started) for download and setup instructions.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@en/identity-server/7.2.0/docs/references/tutorials/connect-with-mattr.md` at
line 182, Replace the vague link text "this documentation" with descriptive text
that explains the destination (for example "MATTR Go Hold getting-started guide"
or "MATTR Go Hold documentation: Getting Started"); update the markdown line
containing the link (the text that currently reads "See [this
documentation](https://learn.mattr.global/docs/holding/go-hold/getting-started)
for details.") so the bracketed text is replaced by the more descriptive phrase
while keeping the same URL.
🤖 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/identity-server/7.2.0/docs/references/tutorials/connect-with-mattr.md`:
- Line 153: Restore the missing hyperlink in the connect-with-mattr.md text by
replacing the plain text "MATTR-supported credential claim mappings" with a
Markdown link pointing to
https://learn.mattr.global/guides/oid4vci/credential-configuration and use
descriptive link text such as "credential configurations" or "claim mappings
guide" where the `claimMappings` parameter is described so readers can follow
the external reference.

---

Nitpick comments:
In `@en/identity-server/7.2.0/docs/references/tutorials/connect-with-mattr.md`:
- Line 182: Replace the vague link text "this documentation" with descriptive
text that explains the destination (for example "MATTR Go Hold getting-started
guide" or "MATTR Go Hold documentation: Getting Started"); update the markdown
line containing the link (the text that currently reads "See [this
documentation](https://learn.mattr.global/docs/holding/go-hold/getting-started)
for details.") so the bracketed text is replaced by the more descriptive phrase
while keeping the same URL.
🪄 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: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 2018dadb-ba0d-4ce6-bd4e-a4eb99f2a907

📥 Commits

Reviewing files that changed from the base of the PR and between d0f90c9 and 371a3a4.

📒 Files selected for processing (1)
  • en/identity-server/7.2.0/docs/references/tutorials/connect-with-mattr.md

Updated the link for claimMappings documentation to point to the credential configurations guide.
@mathushihanratnatheepan
Copy link
Copy Markdown
Author

Thanks for the suggestion! I've updated the link to point to MATTR's current credential configuration documentation.

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.

3 participants