Add multichain error codes #2063
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Pull Request Overview
This PR expands the documentation by adding a table of Multichain API error codes and improves code formatting in markdown pages through SCSS styling updates.
- Added a new "Error codes" section with a table in the multichain API reference markdown.
- Updated the SCSS theme to style inline code, ensuring a consistent look in markdown content.
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| wallet/reference/multichain-api.md | Added a table listing Multichain API error codes along with descriptions and related methods. |
| src/scss/theme/_markdown.scss | Introduced styles for code blocks to use a monospace font with a slightly smaller font size. |
| | Code | Description | Related methods | | ||
| | ---- | ----------- | --------------- | | ||
| | 5000 | Unknown error with request | `wallet_revokeSession`, `wallet_createSession` | |
There was a problem hiding this comment.
@vandan @adonesky1 we're firing -32603 on wallet_revokeSession, same for wallet_createSession
There was a problem hiding this comment.
Hmm. I don't see error codes in that range anywhere in the CAIP standards for this so it seems like we should instead map any errors fired to codes in the 5000 range (ex. 5500, 5501, or 5502 for wallet_revokeSession).
| | 5101 | Requested methods are not supported | `wallet_createSession` | | ||
| | 5102 | Requested notifications are not supported | `wallet_createSession` | | ||
| | 5300 | Invalid `scopedProperties` requested | `wallet_createSession`| |
There was a problem hiding this comment.
we aren't technically firing this yet. thoughts, @adonesky1 ?
| | 5102 | Requested notifications are not supported | `wallet_createSession` | | ||
| | 5300 | Invalid `scopedProperties` requested | `wallet_createSession`| | ||
| | 5301 | `scopedProperties` can only be outside of `sessionScopes` | `wallet_createSession`| |
There was a problem hiding this comment.
i believe we treat this as invalid scope object and simply ignore the scopeobject. thoughts, @adonesky1 ?
|
@jiexi can we supply a fresh list based on the error codes we support? |
Description
Add table with Multichain API error codes.
Preview
https://metamask-docs-q54mtd23a-consensys-ddffed67.vercel.app/wallet/reference/multichain-api/#error-codes
Checklist
Complete this checklist before merging your PR: