Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

imp(ante): depecrate legacy EIP712 ante handler #2078

Merged
merged 4 commits into from Nov 27, 2023

Conversation

facs95
Copy link
Contributor

@facs95 facs95 commented Nov 27, 2023

Description

In the effort of the ante handler refractor, it seems like a good time to deprecate the legacy EIP712 handler


Closes #XXX

@facs95 facs95 marked this pull request as ready for review November 27, 2023 09:49
@facs95 facs95 requested a review from a team as a code owner November 27, 2023 09:49
@facs95 facs95 requested review from Vvaradinov and MalteHerrmann and removed request for a team November 27, 2023 09:49
@facs95 facs95 changed the title depecrate legacy EIP712 ante handler imp(ante): depecrate legacy EIP712 ante handler Nov 27, 2023
Copy link

codecov bot commented Nov 27, 2023

Codecov Report

Merging #2078 (adeeb82) into main (8a46ed8) will increase coverage by 0.05%.
The diff coverage is 0.00%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2078      +/-   ##
==========================================
+ Coverage   70.17%   70.23%   +0.05%     
==========================================
  Files         353      353              
  Lines       25948    25919      -29     
==========================================
- Hits        18210    18205       -5     
+ Misses       6792     6768      -24     
  Partials      946      946              
Files Coverage Δ
app/ante/evm.go 0.00% <ø> (ø)
app/ante/ante.go 44.82% <0.00%> (+4.20%) ⬆️

... and 1 file with indirect coverage changes

Copy link
Contributor

@Vvaradinov Vvaradinov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@fedekunze
Copy link
Contributor

@facs95 can you fix the conflicts 🙏

@fedekunze fedekunze merged commit 651f2a1 into main Nov 27, 2023
33 of 36 checks passed
@fedekunze fedekunze deleted the facs95/deprecate-legacy-eip712 branch November 27, 2023 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants