fix: update blockchain network references from MATIC to POL in docume…#247
fix: update blockchain network references from MATIC to POL in docume…#247manishdex25 wants to merge 4 commits into
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughDocumentation examples, field descriptions, supported-network tables, and tutorial sample outputs were updated to replace the Polygon network identifier from MATIC to POL. ChangesPolygon Network Naming Convention Update
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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
`@docs/introduction/key-components-of-tradetrust/blockchain/supported-network.md`:
- Around line 17-18: The versioned doc still uses the old "MATIC" label for
Polygon entries while the canonical doc uses "POL*" — update the versioned_docs
entry so the rows for chainId 137 and 80002 use "POL*" (and the same chain names
"matic"→"matic" or "amoy"→"amoy" as in the canonical table) to match the current
naming convention, or add a brief note explaining intentional divergence if you
must keep "MATIC"; target the table rows containing the tokens "MATIC" and the
chain IDs 137 and 80002 and replace/update them accordingly.
🪄 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: defaults
Review profile: CHILL
Plan: Pro
Run ID: 6fd22eb9-82dd-4d26-95ff-3d5167d6418b
📒 Files selected for processing (6)
docs/how-tos/contexts.mddocs/how-tos/credential-status.mddocs/how-tos/decentralized-renderer/using-generic-templates.mddocs/introduction/key-components-of-tradetrust/blockchain/supported-network.mddocs/tutorial/creator.mddocs/tutorial/decentralized-renderer.md
👮 Files not reviewed due to content moderation or server errors (1)
- docs/tutorial/decentralized-renderer.md
There was a problem hiding this comment.
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 `@versioned_docs/version-4.x/topics/introduction/supported-network.md`:
- Around line 17-18: The table contains inconsistent "POL*" chain identifiers
(the "POL*" entries for Polygon Mainnet and Polygon Amoy) with an unexplained
asterisk; either remove the asterisk from the "POL*" tokens so they read "POL"
for consistency with other identifiers (e.g., ETH, FREE, ASTRON, XDC), or if the
asterisk is intentional add a short clarifying note/footnote immediately below
the table explaining what "*" means (e.g., indicating rebrand/transition
status), and ensure the note references "POL*" so readers understand which rows
it applies to.
- Around line 17-18: The table shows POL* for chain IDs 137 and 80002 but the
Network Name column contains the value "matic"; confirm whether the "matic" cell
is intended as a network slug or the native token symbol—if it represents the
token, replace "matic" with "POL" to match the rebrand (POL*), otherwise
keep/clarify it as a slug (e.g., "polygon" or "polygon-mainnet") for rows with
chainId 137 and 80002 so the Network Name column is unambiguous and consistent
with the POL migration.
🪄 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: defaults
Review profile: CHILL
Plan: Pro
Run ID: 9100a532-a7e9-4b9e-9f94-7805f29f360d
📒 Files selected for processing (2)
docs/introduction/key-components-of-tradetrust/blockchain/supported-network.mdversioned_docs/version-4.x/topics/introduction/supported-network.md
✅ Files skipped from review due to trivial changes (1)
- docs/introduction/key-components-of-tradetrust/blockchain/supported-network.md
…ntation
Summary by CodeRabbit