Skip to content
This repository has been archived by the owner on Oct 13, 2023. It is now read-only.

Remove Web3Extension Utilities #123

Merged
merged 4 commits into from
Feb 23, 2023
Merged

Remove Web3Extension Utilities #123

merged 4 commits into from
Feb 23, 2023

Conversation

0a1c
Copy link
Contributor

@0a1c 0a1c commented Feb 20, 2023

These utils are no longer necessary. Developers can include an EIP-712 signature directly in the transaction body using createTxRaw from @evmos/proto.

See this PR for more context: evmos/ethermint#1397

Copy link
Contributor

@GAtom22 GAtom22 left a comment

Choose a reason for hiding this comment

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

LGTM! Great work @austinchandra !

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!

@0a1c 0a1c merged commit ffe558d into main Feb 23, 2023
@0a1c 0a1c deleted the austinchandra/transactions-utils branch February 23, 2023 21:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants