Add a pallet to demonstrate calling into a contract #228
Conversation
This reverts commit 89803f0.
ascjones
requested review from
cmichi,
athei,
SkymanOne and
a team
as code owners
| let ink_account_id = | ||
| ink::primitives::AccountId::from(<[u8; 32]>::from(contract.clone())); | ||
| let mut flipper: ink::contract_ref!(Flip, ink::env::DefaultEnvironment) = | ||
| ink_account_id.into(); | ||
| let call_builder = flipper.call_mut(); | ||
|
|
||
| let params = call_builder | ||
| .flip() | ||
| .ref_time_limit(gas_limit.ref_time()) | ||
| .proof_size_limit(gas_limit.proof_size()) | ||
| .params(); | ||
|
|
||
| // Next step is to explore ways to encapsulate the following into the call builder. | ||
| let value = *params.transferred_value(); | ||
| let data = params.exec_input().encode(); | ||
| let weight = Weight::from_parts(params.ref_time_limit(), params.proof_size_limit()); | ||
| let storage_deposit_limit = params.storage_deposit_limit().map(|limit| (*limit).into()); | ||
|
|
||
| let result = pallet_contracts::Pallet::<T>::bare_call( | ||
| who.clone(), | ||
| contract.clone(), | ||
| value.into(), | ||
| weight, | ||
| storage_deposit_limit, | ||
| data, | ||
| pallet_contracts::DebugInfo::Skip, | ||
| pallet_contracts::CollectEvents::Skip, | ||
| pallet_contracts::Determinism::Enforced, | ||
| ) | ||
| .result?; |
There was a problem hiding this comment.
This is the code we want to simplify with a nice API in ink!
pallets/contract-caller/src/lib.rs
Outdated
| pub struct Pallet<T>(_); | ||
|
|
||
| #[pallet::config] | ||
| pub trait Config: frame_system::Config + pallet_contracts::Config { |
There was a problem hiding this comment.
You can just to that here, since contracts inherit from system already
| pub trait Config: frame_system::Config + pallet_contracts::Config { | |
| pub trait Config: pallet_contracts::Config { |
There was a problem hiding this comment.
The #[pallet::config] macro requires the explicit implementation of frame_system::Config
|
Do you think this PR can resolve my question? https://substrate.stackexchange.com/questions/10685/how-to-call-a-contract-from-pallet-with-specific-interface I want to extend my pallet through the developers' uploaded contract, |
Yes, you can use the technique I have used in this PR. I am currently working on a nicer API for this, but it is a few weeks away from being ready. |
That's awesome! Will try when it's landing |
|
Superseded by use-ink/ink#2189, it is more appropriate to live in that repo since that is where the API will live. |
|
This pull request has been mentioned on Polkadot Forum. There might be relevant details there: https://forum.polkadot.network/t/contracts-on-assethub-roadmap/9513/11 |
An foundational step towards use-ink/ink#1674, developing an API for calling into a contract from the runtime.
This PR
contract-callerwhich allows calling into a flipper contractinkto construct the call for the contractflippercontract with a trait and an e2e test which demonstrates it working: instantiates the contract, invokes the pallet, queries the modified state of the contract.The next step (in progress) is to work and iterate on the
inkAPI itself, which will involve aninkPR and a PR here to utilize the new API.