Skip to content

[MOON-2268] update frontier version#2158

Merged
nbaztec merged 1 commit intoperm-runtime-2202from
nish-frontier-fix-eip-3607
Mar 23, 2023
Merged

[MOON-2268] update frontier version#2158
nbaztec merged 1 commit intoperm-runtime-2202from
nish-frontier-fix-eip-3607

Conversation

@nbaztec
Copy link
Copy Markdown
Contributor

@nbaztec nbaztec commented Mar 14, 2023

What does it do?

updates frontier version which includes moonbeam-foundation/frontier#165

⚠️ Breaking Change ⚠️
This fixes the previously erroneous behavior where internal eth calls (eth_call, etc.) were forbidden to be called by accounts having code.

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 nbaztec added B7-runtimenoteworthy Changes should be noted in any runtime-upgrade release notes breaking Needs to be mentioned in breaking changes D5-nicetohaveaudit⚠️ PR contains trivial changes to logic that should be properly reviewed. labels Mar 14, 2023
@nbaztec nbaztec changed the base branch from master to perm-runtime-2202 March 14, 2023 12:50
@nbaztec nbaztec merged commit 0960992 into perm-runtime-2202 Mar 23, 2023
@nbaztec nbaztec deleted the nish-frontier-fix-eip-3607 branch March 23, 2023 11:33
@notlesh notlesh added D1-audited👍 PR contains changes to fund-managing logic that has been properly reviewed and externally audited and removed D5-nicetohaveaudit⚠️ PR contains trivial changes to logic that should be properly reviewed. labels Apr 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

B7-runtimenoteworthy Changes should be noted in any runtime-upgrade release notes breaking Needs to be mentioned in breaking changes D1-audited👍 PR contains changes to fund-managing logic that has been properly reviewed and externally audited

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants