Open b-yap opened 10 months ago
@b-yap the first part is clear to me. For the second part (lacking TokenAllowance
): isn't that a fix required in the runtime or client, i.e., in the pendulum repo instead of in the wrapper contract?
@pendulum-chain/devs is this bug still relevant?
For the second part (lacking TokenAllowance): isn't that a fix required in the runtime or client, i.e., in the pendulum repo instead of in the wrapper contract?
Yes kind of, but it wouldn't hurt to leave a note somewhere that having the currency in the list of allowed_currencies
of the TokenAllowance
pallet is a requirement and you will encounter errors if you don't take care of this before.
Context
The README lacks instructions that are required to make the wrapper contracts work on a runtime.
What's wrong
--target
command argument is outdatedsolang version >= 0.3.1 does not accept
substrate
argument anymore.TODO: it should be
--target polkadot
lacking
TokenAllowance
instructions for testing ERC20WrapperContractTrapped
error will occur when testing the functions using zombienet. It returns 6(orNoProvider
) after callingchain_extension
.TODO
For testing
Depending on which runtime, the chain_spec should add in its GenesisConfig:
^ this means the
Native
currency is the only one supported. To be able to defineTokenAllowanceConfig
in the GenesisConfig, addConfig<T>
ofTokenAllowance
inside theconstruct_runtime!
of the parachain. e.g.In production
Call the
tokenAllowance::addSupportedCurrencies()
extrinsic as a root user.