-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature/doc rpc #83
base: develop
Are you sure you want to change the base?
Feature/doc rpc #83
Conversation
Quality Gate passedIssues Measures |
"methods": [ | ||
{ | ||
"name": "bridge_l1InfoTreeIndexForBridge", | ||
"summary": "Gets the smalles L1 info tree index in which a given bridge is includded", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"summary": "Gets the smalles L1 info tree index in which a given bridge is includded", | |
"summary": "Gets the smallest L1 info tree index in which a given bridge is included", |
"examples": [ | ||
{ | ||
"name": "example", | ||
"description": "", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we remove if empty?
}, | ||
{ | ||
"name": "bridge_injectedInfoAfterIndex", | ||
"summary": "Gets the smalles L1 info tree index in which a given bridge is includded", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same summary as previous?
"result": { | ||
"$ref": "#/components/contentDescriptors/L1InfoTreeLeaf" | ||
}, | ||
"examples": [ ] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should provide example?
"examples": [ ] | ||
}, | ||
{ | ||
"name": "bridge_claimProof", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why claimProof
and not getProof
or just proof
? Looks like too specific
}, | ||
"Metadata": { | ||
"title": "metadata", | ||
"description": "Extra data includded in the bridge", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"description": "Extra data includded in the bridge", | |
"description": "Extra data included in the bridge", |
} | ||
} | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
New line
Description
The file can be rendered here