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: update for eip712 #181

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

chore: update for eip712 #181

wants to merge 3 commits into from

Conversation

leventdem
Copy link
Contributor

WIP work to make the eip712 check working in DecryptionManager.sol for a user decrypt request

@cla-bot cla-bot bot added the cla-signed label Mar 20, 2025
The test is not finalized but cleaned for now.
@jatZama
Copy link
Member

jatZama commented Mar 20, 2025

why did you create a new method instead of modifying createEIP712?

@leventdem
Copy link
Contributor Author

why did you create a new method instead of modifying createEIP712?

I will change this behavior, initially it was because of issue with the chain id which is passed through currying and retrieved from provider.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants