Skip to content

Make IERC7579Execution payable #5410

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
Merged
Amxx merged 1 commit into from
Jan 6, 2025
Merged

Make IERC7579Execution payable #5410

Amxx merged 1 commit into from
Jan 6, 2025

Conversation

@ernestognw
Copy link
Member

@ernestognw ernestognw commented Dec 30, 2024

See ethereum/ERCs#810

PR Checklist

  • Tests
  • Documentation
  • Changeset entry (run npx changeset add)

@changeset-bot
Copy link

changeset-bot bot commented Dec 30, 2024

⚠️ No Changeset found

Latest commit: 501d23d

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@ernestognw ernestognw added this to the 5.2-after-freeze milestone Dec 30, 2024
@Amxx Amxx merged commit bf69b60 into master Jan 6, 2025
20 of 21 checks passed
@Amxx Amxx deleted the ierc7579execution/payable branch January 6, 2025 10:20
@OpenZeppelin OpenZeppelin deleted a comment from 1234336c Jan 6, 2025
Amxx pushed a commit to Amxx/openzeppelin-contracts that referenced this pull request Jan 8, 2025
@ernestognw @Amxx
Make IERC7579Execution payable (OpenZeppelin#5410)
fd981f6 
Signed-off-by: Hadrien Croubois <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Amxx Amxx Amxx approved these changes

@arr00 arr00 arr00 approved these changes

Assignees

No one assigned

Labels

ignore-changeset

Projects

None yet

Milestone

5.2-after-freeze

Development

Successfully merging this pull request may close these issues.

4 participants

@ernestognw @Amxx @arr00