-
Notifications
You must be signed in to change notification settings - Fork 591
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
Add ERC: Structured Data Clear Signing Format #509
base: master
Are you sure you want to change the base?
Conversation
Co-authored-by: Andrew B Coathup <28278242+abcoathup@users.noreply.github.com>
Co-authored-by: Andrew B Coathup <28278242+abcoathup@users.noreply.github.com>
Co-authored-by: Andrew B Coathup <28278242+abcoathup@users.noreply.github.com>
Is it in the scope of this EIP on how to handle off-chain data? e.g. some swap transactions merely contain a reference ( |
Co-authored-by: Usame Algan <5880855+usame-algan@users.noreply.github.com>
Co-authored-by: Usame Algan <5880855+usame-algan@users.noreply.github.com>
Co-authored-by: Usame Algan <5880855+usame-algan@users.noreply.github.com>
Co-authored-by: Usame Algan <5880855+usame-algan@users.noreply.github.com>
I updated to take all reviewers comments into account and pass the linter checks |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Your specification section has parts that begins with "This specification supports..." or "This specification uses...". That phrasing is a little odd. Instead simply describe what requirements implementations have to meet.
The commit b2836af (as a parent of 8544e22) contains errors. |
This is Ledger's proposal for a format enabling smartcontract developers to define the intended UI on wallets when signing a transaction or an EIP 712 message intended for the contract.
For more context, you can refer to this video: https://www.youtube.com/watch?v=-O7aX6vUvs8