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

chore(tests) add EIP712 tx util funcs #1417

Merged
merged 10 commits into from
Feb 22, 2023
Merged

Conversation

GAtom22
Copy link
Contributor

@GAtom22 GAtom22 commented Feb 21, 2023

Description


Closes #XXX

@GAtom22 GAtom22 marked this pull request as ready for review February 21, 2023 19:53
@GAtom22 GAtom22 requested a review from a team as a code owner February 21, 2023 19:53
@GAtom22 GAtom22 requested review from Vvaradinov, facs95, fedekunze, 0a1c and MalteHerrmann and removed request for a team February 21, 2023 19:53
Copy link
Contributor

@fedekunze fedekunze left a comment

Choose a reason for hiding this comment

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

Great work @GAtom22, left some minor suggestions

app/ante/evm/utils_test.go Outdated Show resolved Hide resolved
app/ante/cosmos/utils_test.go Outdated Show resolved Hide resolved
@fedekunze fedekunze enabled auto-merge (squash) February 22, 2023 10:00
@fedekunze fedekunze merged commit 183831f into main Feb 22, 2023
@fedekunze fedekunze deleted the GAtom22/EIP-712-testutil branch February 22, 2023 10:11
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

2 participants