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

Cross chain transactions #586

Open
wants to merge 111 commits into
base: main
Choose a base branch
from

Conversation

codypharm
Copy link

No description provided.

alfheimrShiven and others added 30 commits October 16, 2023 19:16
Removed usage of DeploySmartAccountUtilContracts script since it was buggy.
… AccountGuardianTest till deployment of util contracts soln is found
Account social recovery update: AccountLock as single instance
Account Lock request creation, signature evaluation and execution
Script for deploying all smart-wallet util contracts at once
…unt locking functionality, Chainlink automation commented out(temp)
Sending AccountLock address while initializing Account clones
alfheimrShiven and others added 29 commits December 19, 2023 20:13
Email verification before creating Account recovery request
…ipt for both Anvil & Sepolia, passed email throughout smart account creation chain
Util contracts deploy script updates:

1. Moved smart account creation to deploy script because AccountGuardian & AccountRecovery contracts are deployed only during smart account creation AccountFactory::createAccount(...)

2. Fixed Stack Too Deep error by restricting the return of CCIP contracts.
Note: CCIP contracts will be deployed but not returned as of now.

3. Made the deploy script dynamic for both Anvil & Sepolia chains.

4. Passed the email param used when creating a smart account, across the whole smart account creation chain.

5. Updated the deploy script test.
Account Recovery contract unit test suite
[Fix] `Account::updateAdmin()` function signature and call fix
AccountRecovery unit tests completed. Coverage: 92%
reduced params for commitEmailVerificationHash() and updated test
updated Readme: Problem statement, Architecture
@alfheimrShiven alfheimrShiven deleted the cross-chain-transactions branch January 9, 2024 14:47
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