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

Add ERC: Minimal Modular Smart Accounts #163

Merged
merged 14 commits into from
Jan 30, 2024
Merged

Conversation

zeroknots
Copy link
Contributor

This proposal outlines the minimally required interfaces and behavior for modular smart accounts and modules to ensure interoperability across implementations.

@eip-review-bot
Copy link
Collaborator

eip-review-bot commented Dec 14, 2023

File ERCS/erc-7579.md

Requires 1 more reviewers from @axic, @g11tech, @Pandapip1, @SamWilsn, @xinbenlv

@github-actions github-actions bot added w-ci and removed w-ci labels Dec 14, 2023
@zeroknots zeroknots changed the title Add ERC: Minimal Modular Smart Accounts Add ERC-7579: Minimal Modular Smart Accounts Dec 15, 2023
@github-actions github-actions bot added w-ci and removed w-ci labels Dec 15, 2023
@eip-review-bot eip-review-bot changed the title Add ERC-7579: Minimal Modular Smart Accounts Add ERC: Minimal Modular Smart Accounts Dec 15, 2023
@github-actions github-actions bot added w-ci and removed w-ci labels Dec 15, 2023
@github-actions github-actions bot removed the w-ci label Dec 15, 2023
ERCS/erc-7579.md Outdated Show resolved Hide resolved
ERCS/erc-7579.md Outdated Show resolved Hide resolved
ERCS/erc-7579.md Outdated Show resolved Hide resolved
ERCS/erc-7579.md Outdated Show resolved Hide resolved
ERCS/erc-7579.md Outdated Show resolved Hide resolved
assets/erc-7579/IMSA.sol Outdated Show resolved Hide resolved
ERCS/erc-7579.md Outdated Show resolved Hide resolved
Copy link

The commit 6ac8534 (as a parent of 7298a56) contains errors.
Please inspect the Run Summary for details.

@github-actions github-actions bot added the w-ci label Jan 11, 2024
@github-actions github-actions bot removed the w-ci label Jan 11, 2024
@kopy-kat
Copy link
Contributor

thank you for the feedback @SamWilsn, its been incorporated so it would be great if you could look over it again and merge if it looks good to you

@derekchiang
Copy link

Hi @SamWilsn just a gentle poke to review and merge this if it looks good.

@lightclient lightclient enabled auto-merge (squash) January 30, 2024 00:13
@lightclient lightclient merged commit 291f41e into ethereum:master Jan 30, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants