Skip to content

Update ERC-7943: Split into separate interfaces#1210

Merged
eip-review-bot merged 4 commits intoethereum:masterfrom
xaler5:patch-1
Sep 22, 2025
Merged

Update ERC-7943: Split into separate interfaces#1210
eip-review-bot merged 4 commits intoethereum:masterfrom
xaler5:patch-1

Conversation

@xaler5
Copy link
Copy Markdown
Contributor

@xaler5 xaler5 commented Sep 12, 2025

  • Documented that canTransfer should return false on paused contracts.
  • Documented discrepancy of canTransfer performing isUserAllowed with ERC-3643.
  • Split into separate interfaces (fungible, non fungible, multi token)
  • Refactored examples
  • Changed the "should not allow freezing more assets than balance" to a "should be able to freeze more than that"

Things to review:

  • getFrozenTokens for ERC-721 should return uint256 or a bool ?

from ETH Magicians discussion, it is good to leave it as bool.

  • Errors should be named the same even if with different parameters across different interfaces ?

Error renamed in the non-fungible case

  • Review correctness of provided examples

At least one confirmation that those are good.

@eip-review-bot
Copy link
Copy Markdown
Collaborator

eip-review-bot commented Sep 12, 2025

✅ All reviewers have approved.

@eip-review-bot eip-review-bot changed the title [Update 7943] Split into separate interfaces Update ERC-7943: [Update 7943] Split into separate interfaces Sep 12, 2025
@eip-review-bot eip-review-bot enabled auto-merge (squash) September 12, 2025 08:04
eip-review-bot
eip-review-bot previously approved these changes Sep 12, 2025
Copy link
Copy Markdown
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...

@xaler5 xaler5 changed the title Update ERC-7943: [Update 7943] Split into separate interfaces Update ERC-7943: Split into separate interfaces Sep 12, 2025
@xaler5 xaler5 marked this pull request as draft September 12, 2025 08:06
auto-merge was automatically disabled September 12, 2025 08:06

Pull request was converted to draft

@xaler5 xaler5 marked this pull request as ready for review September 22, 2025 11:13
@eip-review-bot eip-review-bot enabled auto-merge (squash) September 22, 2025 11:14
eip-review-bot
eip-review-bot previously approved these changes Sep 22, 2025
Copy link
Copy Markdown
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...

auto-merge was automatically disabled September 22, 2025 11:20

Head branch was pushed to by a user without write access

@eip-review-bot eip-review-bot enabled auto-merge (squash) September 22, 2025 11:21
Copy link
Copy Markdown
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...

Copy link
Copy Markdown
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 0ad86e6 into ethereum:master Sep 22, 2025
10 of 14 checks passed
@xaler5 xaler5 deleted the patch-1 branch September 22, 2025 11:43
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