Skip to content
This repository has been archived by the owner on Sep 30, 2024. It is now read-only.

Recovery hardening #234

Closed
wants to merge 61 commits into from
Closed

Recovery hardening #234

wants to merge 61 commits into from

Conversation

JohnGuilding
Copy link
Contributor

What is this PR doing?

How can these changes be manually tested?

Does this PR resolve or contribute to any issues?

Checklist

  • I have manually tested these changes
  • Post a link to the PR in the group chat

Guidelines

  • If your PR is not ready, mark it as a draft
  • The resolve conversation button is for reviewers, not authors
    • (But add a 'done' comment or similar)

jacque006 and others added 30 commits April 2, 2024 22:43
Bump circom & other deps versions in zkp package (unrelated to ether-email-auth)
Use yarn create vite w/ React, Typescript
Add WalletConnect provider lib
Add WalletConnect modal to connect to safe app.
Add basic UI to showcase flow, with stubbed out relayer calls.
Remove v4 OpenZepplin, email auth v5 branch
Remove old, stale deps.
Add submodules step to package.json, README for setup.
Add initial base sepolia configs.
Fix forge build OZ v5 ECDSA -> MessageHashUtils.
Update README.
Switch to ConnectKit + wagmi + viem stack.
Breakout to multiple react components.
Fill out all relayer HTTP endpoints & types.
Add ethers.js for function encoding, other utils if needed.
Add SimpleWallet deploy to aid in testing of relayer.
jacque006 and others added 28 commits April 6, 2024 17:09
Add recovery router to deploy script, rename.
Update relayer with more type defs.
Add GH action to deploy app.
Add snarkjslib to aid in generating account code.
Fix TS typing.
Add account code util func.
Update recovery plugin to allow acces to router.
Remove multicalls as we can get them from wagmi config.
Add utils for account code & guardian address.
Add nodejs polyfill for vite.
Add wagmi config to link contracts in future.
Remove router deploy from script.
@jacque006
Copy link
Contributor

@JohnGuilding Can this be closed? Is there anything we need to extract?

@jacque006 jacque006 closed this Jul 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants