Conversation
|
| const { amount, sender, source } = ctx; | ||
| const transfer = await this.getTransfer(ctx); | ||
| return this.adapter.buildCall(sender, amount, source.feeBalance, transfer); | ||
| const callType = route.contract ? CallType.Evm : CallType.Substrate; |
There was a problem hiding this comment.
what about programs / moves ? this line is fishy ..
| const address = route.contract | ||
| ? await formatEvmAddress(sender, chain) | ||
| : sender; | ||
| const feeCallType = route.contract ? CallType.Evm : CallType.Substrate; |
| if (chain.isEvmParachain()) { | ||
| const evmParachain = chain as EvmParachain; | ||
| return evmParachain.getDerivatedAddress(address); | ||
| return callType === CallType.Evm |
There was a problem hiding this comment.
It has isEvmParachain check so it only has evm and substrate options
| const transfer = await this.getTransfer(ctx); | ||
| return this.adapter.buildCall(sender, amount, source.feeBalance, transfer); | ||
| const callType = route.contract ? CallType.Evm : CallType.Substrate; | ||
| const account = await resolveAddress(sender, chain, callType); |
There was a problem hiding this comment.
besides i would move resolve logic to getTransfer where builders are
2 participants
Add bidirectional address resolution driven by
CallType- users can now pass either H160 or SS58 addressesissue #134