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

Add MIP for eth_sign discontinuation #42

Merged
merged 7 commits into from
May 22, 2024
Merged

Conversation

vandan
Copy link
Collaborator

@vandan vandan commented May 15, 2024

Add an initial MIP for eth_sign discontinuation

Add an initial MIP for eth_sign deprecation
MIPs/mip-eth-sign.md Outdated Show resolved Hide resolved
MIPs/mip-eth-sign.md Outdated Show resolved Hide resolved
vandan and others added 3 commits May 15, 2024 16:20
Co-authored-by: Alex Donesky <[email protected]>
Adjust the description in the Motivation section.

Co-authored-by: Alex Donesky <[email protected]>
Update language to be about discontinuation because the method had already been considered deprecated last year.
@vandan vandan changed the title Add MIP for eth_sign deprecation Add MIP for eth_sign discontinuation May 17, 2024
Copy link
Collaborator

@adonesky1 adonesky1 left a comment

Choose a reason for hiding this comment

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

LGTM

Adds a link back to docs with coverage of the alternative signing methods.
MIPs/mip-eth-sign.md Outdated Show resolved Hide resolved
vandan and others added 2 commits May 22, 2024 12:07
Update with a more descriptive explanation.

Co-authored-by: Shane <[email protected]>
Highlight the potential for future risks

Co-authored-by: Shane <[email protected]>
@vandan vandan merged commit dd6bf1f into main May 22, 2024
2 checks passed
@vandan vandan deleted the vandan/deprecate-eth-sign branch May 22, 2024 20:14
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