-
Notifications
You must be signed in to change notification settings - Fork 26
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
feat: add erc7484 to msaadvanced #26
Conversation
alternate way would also be to have this .check happen inside a hook |
yep, we have a hook example here: https://github.com/rhinestonewtf/modulekit/blob/examples/audit-ready/examples/src/RegistryHook/RegistryHook.sol |
one thing that I thought could be useful is demonstrate more advanced integrations in msa advanced to highlight some additional features that are outside of 7579 but accounts could use (eg hooks, 7484, policy/signers) - wdyt |
yes I do want to add tests in 7579 repo (reference / biconomy v3 -calling this 'Nexus' now) using module-kit
|
No description provided.