-
Notifications
You must be signed in to change notification settings - Fork 20
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
doc: w3n query #261
doc: w3n query #261
Conversation
This pull request is automatically being deployed by Amplify Hosting (learn more). |
* **submitterAccount**: The keyring pair of the submitter. | ||
* **signCallback**: The sign extrinsic callback function. This function is used to sign the extrinsic, read more that in [the SignCallback section](../07_signCallback.md). | ||
|
||
The function `releaseWeb3Name` uses the KILT SDK to create a *web3name release transaction* using `api.tx.web3Names.releaseByOwner`. |
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.
Is this the right direction to go? We would not be able to catch any of these inconsistencies if the naming changes, at some point. What about adding comments to the code snippet directly?
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.
I think adding comments to the code make it less readable. e.g. hard to spot what is actually going on, finding the function calls in between all the comments.
Yes this can get outdated, but I would hope that not only the code example is checked when there is an error, but also the guide where the code is used. 😓
</TsJsBlock> | ||
|
||
In the code example above, the `queryDidDocument` function takes a web3Name parameter, which represents the web3name to be resolved. | ||
It internally uses the `api.call.did.queryByWeb3Name` method to query the information of the provided web3name from the blockchain. |
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 as above.
Co-authored-by: Antonio <[email protected]>
Co-authored-by: Antonio <[email protected]>
fixes https://github.com/KILTprotocol/ticket/issues/2780
🤖