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 ERC: Transfer With Authorization #598

Merged

Conversation

dongri
Copy link
Contributor

@dongri dongri commented Aug 21, 2024

Ref: #504

@eip-review-bot
Copy link
Collaborator

eip-review-bot commented Aug 21, 2024

✅ All reviewers have approved.

@eip-review-bot eip-review-bot changed the title Add erc-transfer-with-authorization Add ERC: Transfer With Authorization Aug 21, 2024
@github-actions github-actions bot added the w-ci label Aug 21, 2024
@@ -0,0 +1,536 @@
---
eip: 3009
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
eip: 3009
eip: 7758

Assigning next sequential EIP/ERC/RIP number.

Please also update the filename.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@abcoathup Thank you for the review
ea63268

  • Update eip number to 7758
  • Change filename

@abcoathup
Copy link
Contributor

Please close ethereum/EIPs#8819

* Change filename
@dongri dongri requested a review from abcoathup August 22, 2024 02:11
ERCS/erc-7758.md Outdated
eip: 7758
title: Transfer With Authorization
author: Peter Jihoon Kim (@petejkim), Kevin Britz (@kbrizzle), David Knott (@DavidLKnott), Dongri Jin (@dongri)
discussions-to: https://github.com/ethereum/EIPs/issues/3010
Copy link
Contributor

Choose a reason for hiding this comment

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

Please create a discussions topic in Eth Magicians. It just needs to contain a link to this PR.
https://ethereum-magicians.org/

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I signed up, but it seems I can't post yet. I'll add the link once I can post.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@dongri dongri requested a review from abcoathup August 22, 2024 12:01
Copy link

The commit 1640548 (as a parent of c63987b) contains errors.
Please inspect the Run Summary for details.

@github-actions github-actions bot removed the w-ci label Aug 27, 2024
@eip-review-bot eip-review-bot enabled auto-merge (squash) August 27, 2024 21:10
Copy link
Collaborator

@eip-review-bot eip-review-bot left a comment

Choose a reason for hiding this comment

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

All Reviewers Have Approved; Performing Automatic Merge...

@eip-review-bot eip-review-bot merged commit 22ace52 into ethereum:master Aug 27, 2024
10 of 11 checks passed
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.

5 participants