-
Couldn't load subscription status.
- Fork 0
Update docs for smart contracts #12
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
base: main
Are you sure you want to change the base?
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
| #[ink::contract(env = IDNEnvironment)] | ||
| pub mod MyContract { | ||
| // make the custom Environment callable | ||
| use crate::IDNEnvironment; |
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.
they still need to make the IDNEnvironment available when deployed on the IDN.
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 added this portion back. I removed it because in the example rand-extension contract we do not have this.
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.
ahh yeah, it's a little nuanced. If they use the shuffle/select then it's already baked in.
|
|
||
| ➜ Repository for the [idn-client-contract-lib](https://github.com/ideal-lab5/idn-sdk/tree/main/contracts/idn-client-contract-lib). | ||
|
|
||
| > ⚠️ This library has not yet been published. |
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.
The lib still hasn't been published (to crates.io).
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 added this comment back so users know that it hasn't been published yet.
| pub fn new() -> Self { | ||
| Self { | ||
| idn_client: IdnClient::new( | ||
| idn_para_id, // IDN parachain ID |
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.
we should define these values explicitly in an example.
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 updated them with example values.
…mment related to max xcm fees
…o rand contract from quick start page
…k for HRMP channel pre-req. Shorten Quick Start section related to cross chain smart contract
…gmt page for x chain smart contract.
…n for sub management related info. indent contents of nubmered lists
No description provided.