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

Bringing in the spec and the interface #40

Merged
merged 19 commits into from
Apr 23, 2024
Merged

Bringing in the spec and the interface #40

merged 19 commits into from
Apr 23, 2024

Conversation

alcueca
Copy link
Owner

@alcueca alcueca commented Apr 22, 2024

After merging this, I'll rename the repo.

I could also bring the reference implementation from alcueca/erc7399-reference, if it seems useful.

/// @param data The ABI encoded user data
/// @param callback The address and signature of the callback function
/// @return result ABI encoded result of the callback
function flash(
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we add the vyper compatible function?

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you want, be my guest.

@alcueca alcueca changed the base branch from gnosis-safe to main April 23, 2024 09:11
@alcueca alcueca merged commit 42f4d94 into main Apr 23, 2024
3 checks passed
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.

2 participants