Skip to content

Conversation

@FrederikBolding
Copy link
Member

@FrederikBolding FrederikBolding commented Dec 19, 2024

Unblock all variants of eth_signTypedData as they are deemed lower risk following UI changes on the clients.

Also adds logic to use it in the Ethereum provider example Snap.

@codecov
Copy link

codecov bot commented Dec 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.71%. Comparing base (3e615f5) to head (c665c07).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2969   +/-   ##
=======================================
  Coverage   94.71%   94.71%           
=======================================
  Files         497      497           
  Lines       10802    10802           
  Branches     1649     1649           
=======================================
  Hits        10231    10231           
  Misses        571      571           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@FrederikBolding FrederikBolding force-pushed the fb/unblock-signtypeddata branch from e4beb01 to cf30710 Compare December 19, 2024 15:15
@FrederikBolding FrederikBolding marked this pull request as ready for review December 19, 2024 15:15
@FrederikBolding FrederikBolding requested a review from a team as a code owner December 19, 2024 15:15
@FrederikBolding FrederikBolding force-pushed the fb/unblock-signtypeddata branch from cf30710 to c665c07 Compare January 6, 2025 10:50
@FrederikBolding FrederikBolding merged commit 296244d into main Jan 6, 2025
166 checks passed
@FrederikBolding FrederikBolding deleted the fb/unblock-signtypeddata branch January 6, 2025 11:01
Mrtenz pushed a commit that referenced this pull request Jan 8, 2025
Unblock all variants of `eth_signTypedData` as they are deemed lower
risk following UI changes on the clients.

Also adds logic to use it in the Ethereum provider example Snap.
PatrykLucka pushed a commit that referenced this pull request Jan 13, 2025
Unblock all variants of `eth_signTypedData` as they are deemed lower
risk following UI changes on the clients.

Also adds logic to use it in the Ethereum provider example Snap.
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.

3 participants