Skip to content

Adds ERC1155TransferEnforcer #124

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Adds ERC1155TransferEnforcer #124

wants to merge 8 commits into from
+1,202 −0

Conversation

hanzel98
Copy link
Contributor

@hanzel98 hanzel98 commented Jun 10, 2025 •
edited
Loading

What?

  • This ERC1155TransferEnforcer is needed to validate the transfer amounts and ids for erc1155.

Why?

  • We need this enforcer to create delegations for specific amounts and ids on erc1155

How?

  • Creating amount limits for each token, it validates and stores the used amount in a spent map

@hanzel98 hanzel98 self-assigned this Jun 10, 2025
@hanzel98 hanzel98 requested a review from a team as a code owner June 10, 2025 03:12
@hanzel98 hanzel98 marked this pull request as draft June 10, 2025 03:12
MoMannn
MoMannn requested changes Jun 10, 2025
View reviewed changes
MoMannn
MoMannn requested changes Jun 10, 2025
View reviewed changes
@hanzel98 hanzel98 changed the title feat: adds ERC1155TransferEnforcer Adds ERC1155TransferEnforcer Jun 13, 2025
@hanzel98 hanzel98 requested a review from MoMannn June 13, 2025 20:28
@hanzel98 hanzel98 marked this pull request as ready for review June 13, 2025 20:32
MoMannn
MoMannn previously approved these changes Jun 17, 2025
View reviewed changes
@hanzel98 hanzel98 requested a review from MoMannn June 19, 2025 20:28
@hanzel98 hanzel98 requested a review from McOso June 19, 2025 20:29
@hanzel98 hanzel98 added the new release This PR goes to the next release label Jun 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MoMannn MoMannn Awaiting requested review from MoMannn

@McOso McOso Awaiting requested review from McOso

At least 1 approving review is required to merge this pull request.

Assignees

@hanzel98 hanzel98

Labels
new release This PR goes to the next release
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hanzel98 @MoMannn