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

Local aa infra without foundry chain #637

Merged
merged 12 commits into from
Jul 10, 2024

Conversation

KannuSingh
Copy link
Collaborator

@KannuSingh KannuSingh commented Jul 10, 2024

This PR removes the toggle enable local AA and instead checks the .env values for NEXT_PUBLIC_LOCAL_BUNDLER_URL, NEXT_PUBLIC_LOCAL_PAYMASTER_URL, and NEXT_PUBLIC_LOCAL_CLIENT_URL to override the respective URLs.

The previous implementation(#634) assumed

  • The chainId of the local anvil instance would be 31337 (Foundry chain), but this is only the case if we run an anvil instance without forking any chain. For example, if we fork the Sepolia testnet, the chainId of the local running anvil chain will also be that of the Sepolia chain. This removes the need to use Foundry as a separate chain.
  • This will also be tested on deployed instance of sample wallet(which is not the case)

For more information on how to set up local testing, refer to how to setup local-testing.

When running the project locally and wanting to use the local forked network and AA setup, just add the .env variable values, and the project will automatically use the local setup:

NEXT_PUBLIC_LOCAL_BUNDLER_URL=
NEXT_PUBLIC_LOCAL_PAYMASTER_URL=
NEXT_PUBLIC_LOCAL_CLIENT_URL=

Copy link

vercel bot commented Jul 10, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
decentralized-relay-wallet ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 10, 2024 3:28pm
react-wallet-v2 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 10, 2024 3:28pm
10 Skipped Deployments
Name Status Preview Comments Updated (UTC)
decentralized-relay-app ⬜️ Ignored (Inspect) Visit Preview Jul 10, 2024 3:28pm
malicious-dapp-verify-simulation ⬜️ Ignored (Inspect) Visit Preview Jul 10, 2024 3:28pm
react-auth-dapp ⬜️ Ignored (Inspect) Visit Preview Jul 10, 2024 3:28pm
react-auth-wallet ⬜️ Ignored (Inspect) Visit Preview Jul 10, 2024 3:28pm
react-dapp-v2 ⬜️ Ignored (Inspect) Visit Preview Jul 10, 2024 3:28pm
react-dapp-v2-cosmos-provider ⬜️ Ignored (Inspect) Visit Preview Jul 10, 2024 3:28pm
react-dapp-v2-with-ethers ⬜️ Ignored (Inspect) Visit Preview Jul 10, 2024 3:28pm
react-dapp-v2-with-web3js ⬜️ Ignored (Inspect) Visit Preview Jul 10, 2024 3:28pm
svelte-web3modal ⬜️ Ignored (Inspect) Visit Preview Jul 10, 2024 3:28pm
vue-dapp-auth ⬜️ Ignored (Inspect) Visit Preview Jul 10, 2024 3:28pm

@KannuSingh KannuSingh merged commit 32319e1 into main Jul 10, 2024
19 checks passed
@KannuSingh KannuSingh deleted the local-aa-infra-without-foundry-chain branch July 10, 2024 16:02
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