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 EIP-5883: Token Transfer by Social Recovery #5883

Merged
merged 20 commits into from
Feb 24, 2023
Merged

Conversation

wsdt
Copy link
Contributor

@wsdt wsdt commented Nov 6, 2022

title: Token Transfer by Social Recovery

description: This EIP describes a token recovery by a transfer from an identity which is not accessible anymore to another newly created identity by social interaction.

Looking forward to feedback!

@wsdt wsdt requested a review from eth-bot as a code owner November 6, 2022 10:24
@eth-bot
Copy link
Collaborator

eth-bot commented Nov 6, 2022

All reviewers have approved. Auto merging...

@abcoathup
Copy link
Contributor

I couldn't see where an EIP editor assigned the EIP 5807?
I assume this EIP should be EIP5882 as that was the original PR (unclear why #5882 was closed)

EIPS/eip-5807.md Outdated Show resolved Hide resolved
Dankyybomb1
Dankyybomb1 previously approved these changes Nov 9, 2022
Co-authored-by: Sam Wilson <57262657+SamWilsn@users.noreply.github.com>
@github-actions github-actions bot added c-new Creates a brand new proposal s-draft This EIP is a Draft t-erc labels Nov 9, 2022
@wsdt
Copy link
Contributor Author

wsdt commented Nov 9, 2022

Does anybody know why the HTMLProofer fails, receiving this error

Liquid Exception: Liquid syntax error (line 33): Variable '{{{\displaystyle\sum_{i = 1}' was not properly terminated with regexp: /\}\}/ in EIPS/eip-5883.md

That would be in the middle of the code snippet.

@Pandapip1 Pandapip1 changed the title EIP - Social Recovery Add EIP-5883: Token Transfer by Social Recovery Nov 9, 2022
@mrqc
Copy link

mrqc commented Nov 13, 2022

We checked the markdown linter issues for now. The html proofer seems to be wrong.

@Pandapip1
Copy link
Member

HTMLproofer is not wrong. Make sure that you escape any {{ or }}.

@wsdt
Copy link
Contributor Author

wsdt commented Nov 16, 2022

Should be fine now, thanks everyone!

[@axic, @SamWilsn, @Pandapip1]

@wsdt
Copy link
Contributor Author

wsdt commented Nov 25, 2022

When does the eth-bot review the PR? Is there something else to be done?
Thank you in advance!

[@axic, @SamWilsn, @Pandapip1, @eth-bot ]

EIPS/eip-5883.md Outdated Show resolved Hide resolved
EIPS/eip-5883.md Outdated Show resolved Hide resolved
EIPS/eip-5883.md Outdated Show resolved Hide resolved
EIPS/eip-5883.md Show resolved Hide resolved
EIPS/eip-5883.md Outdated Show resolved Hide resolved
EIPS/eip-5883.md Outdated Show resolved Hide resolved
EIPS/eip-5883.md Outdated Show resolved Hide resolved
EIPS/eip-5883.md Show resolved Hide resolved
EIPS/eip-5883.md Outdated Show resolved Hide resolved
EIPS/eip-5883.md Outdated Show resolved Hide resolved
EIPS/eip-5883.md Outdated Show resolved Hide resolved
@github-actions
Copy link

There has been no activity on this pull request for 2 weeks. It will be closed after 3 months of inactivity. If you would like to move this PR forward, please respond to any outstanding feedback or add a comment indicating that you have addressed all required feedback and are ready for a review.

@github-actions github-actions bot added the w-stale Waiting on activity label Jan 11, 2023
@wsdt
Copy link
Contributor Author

wsdt commented Jan 11, 2023

Ping - any chance to get this to Draft? Are we missing something?

@github-actions github-actions bot removed the w-stale Waiting on activity label Jan 16, 2023
@ethereum ethereum deleted a comment Jan 31, 2023
@SamWilsn
Copy link
Contributor

There's still a few outstanding comments. Most importantly about the Rationale section.

@wsdt wsdt dismissed a stale review via 11f3be9 January 31, 2023 10:28
@wsdt
Copy link
Contributor Author

wsdt commented Jan 31, 2023

Fixed, sorry overlooked them!
Thanks @SamWilsn

@wsdt wsdt requested review from SamWilsn, xinbenlv and Dankyybomb1 and removed request for SamWilsn and xinbenlv February 1, 2023 19:03
@wsdt
Copy link
Contributor Author

wsdt commented Feb 5, 2023

@eth-bot

@wsdt
Copy link
Contributor Author

wsdt commented Feb 15, 2023

anything missing? Would appreciate an update, thank you :-)

Copy link
Member

@Pandapip1 Pandapip1 left a comment

Choose a reason for hiding this comment

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

LGTM. You might have outdated CI - I recommend rebasing your branch (and not using your fork's master branch to submit PRs!)

@eth-bot eth-bot added e-consensus Waiting on editor consensus e-review Waiting on editor to review labels Feb 24, 2023
@eth-bot eth-bot enabled auto-merge (squash) February 24, 2023 19:02
@eth-bot eth-bot merged commit 8dd93a9 into ethereum:master Feb 24, 2023
fulldecent pushed a commit to fulldecent/EIPs that referenced this pull request Mar 13, 2023
* EIP - Social Recovery

* delimiter

* Social Recovery - Validation errors fix

* Update EIPS/eip-5807.md

Co-authored-by: Sam Wilson <57262657+SamWilsn@users.noreply.github.com>

* EIP 5883 - fixing validation errors

* final validation errors fixed - EIP 5883

* Update eip-5883.md

* Fix Markdown linter

* Update eip-5883.md

* Update eip-5883.md

* Update eip-5883.md

* Update EIPS/eip-5883.md

Co-authored-by: Sam Wilson <57262657+SamWilsn@users.noreply.github.com>

* Update EIPS/eip-5883.md

Co-authored-by: Sam Wilson <57262657+SamWilsn@users.noreply.github.com>

* Update EIPS/eip-5883.md

Co-authored-by: Sam Wilson <57262657+SamWilsn@users.noreply.github.com>

* Update EIPS/eip-5883.md

Co-authored-by: Sam Wilson <57262657+SamWilsn@users.noreply.github.com>

* Update EIPS/eip-5883.md

Co-authored-by: Sam Wilson <57262657+SamWilsn@users.noreply.github.com>

* Update EIPS/eip-5883.md

Co-authored-by: Sam Wilson <57262657+SamWilsn@users.noreply.github.com>

* applied feedback - 5883

* Fixing comments.

---------

Co-authored-by: Sam Wilson <57262657+SamWilsn@users.noreply.github.com>
axelcabee pushed a commit to axelcabee/EIPs that referenced this pull request May 6, 2023
* EIP - Social Recovery

* delimiter

* Social Recovery - Validation errors fix

* Update EIPS/eip-5807.md

Co-authored-by: Sam Wilson <57262657+SamWilsn@users.noreply.github.com>

* EIP 5883 - fixing validation errors

* final validation errors fixed - EIP 5883

* Update eip-5883.md

* Fix Markdown linter

* Update eip-5883.md

* Update eip-5883.md

* Update eip-5883.md

* Update EIPS/eip-5883.md

Co-authored-by: Sam Wilson <57262657+SamWilsn@users.noreply.github.com>

* Update EIPS/eip-5883.md

Co-authored-by: Sam Wilson <57262657+SamWilsn@users.noreply.github.com>

* Update EIPS/eip-5883.md

Co-authored-by: Sam Wilson <57262657+SamWilsn@users.noreply.github.com>

* Update EIPS/eip-5883.md

Co-authored-by: Sam Wilson <57262657+SamWilsn@users.noreply.github.com>

* Update EIPS/eip-5883.md

Co-authored-by: Sam Wilson <57262657+SamWilsn@users.noreply.github.com>

* Update EIPS/eip-5883.md

Co-authored-by: Sam Wilson <57262657+SamWilsn@users.noreply.github.com>

* applied feedback - 5883

* Fixing comments.

---------

Co-authored-by: Sam Wilson <57262657+SamWilsn@users.noreply.github.com>
GAEAlimited pushed a commit to GAEAlimited/EIPs that referenced this pull request Jun 19, 2024
* EIP - Social Recovery

* delimiter

* Social Recovery - Validation errors fix

* Update EIPS/eip-5807.md

Co-authored-by: Sam Wilson <57262657+SamWilsn@users.noreply.github.com>

* EIP 5883 - fixing validation errors

* final validation errors fixed - EIP 5883

* Update eip-5883.md

* Fix Markdown linter

* Update eip-5883.md

* Update eip-5883.md

* Update eip-5883.md

* Update EIPS/eip-5883.md

Co-authored-by: Sam Wilson <57262657+SamWilsn@users.noreply.github.com>

* Update EIPS/eip-5883.md

Co-authored-by: Sam Wilson <57262657+SamWilsn@users.noreply.github.com>

* Update EIPS/eip-5883.md

Co-authored-by: Sam Wilson <57262657+SamWilsn@users.noreply.github.com>

* Update EIPS/eip-5883.md

Co-authored-by: Sam Wilson <57262657+SamWilsn@users.noreply.github.com>

* Update EIPS/eip-5883.md

Co-authored-by: Sam Wilson <57262657+SamWilsn@users.noreply.github.com>

* Update EIPS/eip-5883.md

Co-authored-by: Sam Wilson <57262657+SamWilsn@users.noreply.github.com>

* applied feedback - 5883

* Fixing comments.

---------

Co-authored-by: Sam Wilson <57262657+SamWilsn@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c-new Creates a brand new proposal e-consensus Waiting on editor consensus e-review Waiting on editor to review s-draft This EIP is a Draft t-erc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants