moonbeam icon indicating copy to clipboard operation
moonbeam copied to clipboard

[MOON-863] add full proxy precompile (discussion)

Open nbaztec opened this issue 2 years ago • 2 comments

What does it do?

adds proxy precompile

This PR is for discussion purposes on what to support in the proxy precompile after https://github.com/PureStake/moonbeam/pull/1705.

What important points reviewers should know?

Is there something left for follow-up PRs?

What alternative implementations were considered?

Are there relevant PRs or issues in other repositories (Substrate, Polkadot, Frontier, Cumulus)?

What value does it bring to the blockchain users?

nbaztec avatar Jul 13 '22 17:07 nbaztec

All proxy precompile functions that take an encoded call as parameter should take bounded inputs, like this PR: https://github.com/PureStake/moonbeam/pull/1656

librelois avatar Jul 21 '22 11:07 librelois

I'll wait for https://github.com/PureStake/moonbeam/pull/1656 to get merged, since it contains the necessary BoundedVec Read implementations.

nbaztec avatar Jul 21 '22 13:07 nbaztec

@nbaztec Can now be updated to BoundedVec and precompile macro :)

nanocryk avatar Oct 21 '22 12:10 nanocryk

Closing in favor of https://github.com/PureStake/moonbeam/pull/2022

nbaztec avatar Jan 16 '23 09:01 nbaztec