[Bug] capabilities field incorrectly shown as array in ERC-5792 code examples
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 85/100
- Issue type
- Bug
- Clarity
- Clearly specified
- Activity status
- Quiet
- Tech stack
- javascript
- Domain
- blockchain, documentation
Research direction
Open tmp-builder-codes-outline.mdx and search all code examples for capabilities: [{ dataSuffix }]. Compare each occurrence with the ERC-5792 object format described in the issue. Done means every capabilities example uses the object form with the erc8021 key and no array form remains.
Written by the indexing model from the issue text.
Description
Description
In tmp-builder-codes-outline.mdx, the capabilities field is used as an
array in all code examples:
capabilities: [{ dataSuffix }]
However, the ERC-5792 spec defines capabilities as an object (key-value
map), not an array. The correct format should be:
capabilities: {
"erc8021": { dataSuffix }
}
Location
tmp-builder-codes-outline.mdx — multiple code examples throughout the file
Impact
- Developers following these examples will produce invalid ERC-5792 calls
- Wallets that strictly follow the spec will reject these calls
- Could cause silent attribution failures
Suggested Fix
Update all capabilities examples to use the correct object format as
defined in the ERC-5792 specification.
- Dominant language
- JavaScript
- Stars
- 337
- Forks
- 798
- Avg merge
- 7h 24m
- Merged PRs (30d)
- 51
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from base/docs
-
Difficulty 1/5 Under an hour Newbie friendliness 95/100
-
Difficulty 1/5 Under an hour Newbie friendliness 95/100
-
Difficulty 1/5 Under an hour Newbie friendliness 95/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
HarperFast/skills#96 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Automattic/studio#4908 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 86/100
sugarlabs/musicblocks#8847 ·
-
client-controller-update ta-bot-triage team-money-movement
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
MetaMask/metamask-mobile#36594 ·