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

MacroForwarderERC712 Version #1798

Closed
wants to merge 6 commits into from
Closed

Conversation

hellwolf
Copy link
Contributor

@hellwolf hellwolf commented Jan 22, 2024

Copy link

Changelog Reminder

Reminder to update the CHANGELOG.md for any of the modified packages in this PR.

  • CHANGELOG.md modified
  • Double check before merge

@hellwolf hellwolf changed the base branch from dev to feature-trusted-macros January 22, 2024 18:38
@hellwolf hellwolf linked an issue Jan 22, 2024 that may be closed by this pull request
8 tasks
@hellwolf hellwolf changed the base branch from feature-trusted-macros to dev January 23, 2024 08:33
@hellwolf hellwolf changed the title Trusted Macros EIP-712 Version MacroForwarder EIP-712 Version Feb 16, 2024
@hellwolf hellwolf changed the title MacroForwarder EIP-712 Version MacroForwarderERC712 Version Feb 16, 2024
@hellwolf
Copy link
Contributor Author

hellwolf commented Oct 9, 2024

Instead, the new idiomatic way forward is that eip-712 support macro should simply be built as part of the regular macro.

See SuperBoring example: https://github.com/superfluid-finance/averageX/pull/219

@hellwolf hellwolf closed this Oct 9, 2024
@hellwolf hellwolf deleted the feature-trusted-macros-eip712 branch October 9, 2024 10:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ETHEREUM-CONTRACTS] TrustedMacros: A TrustedForwarder that rules them all
2 participants