-
Notifications
You must be signed in to change notification settings - Fork 65
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
Autoprompt the user to switch network in the connect button #288
Comments
Hey @aguzmant103 I've got part of the fix. We send in the supported chains as 5. Basically, this means the user can't hook up their wallet unless they're on Goerli. Though, I'm not totally sure if the user experience will be great with this. By the way, we're using @web-react/core v6. |
Do you have a working fork to test? I initially feel that's still an acceptable UX but dunno, @vplasencia what do you think? |
Bandada now supports only the Goerli network. If you try to use it on any other network, it will log an error. fix bandada-infra#288
Code : https://github.com/SrinivasJoshi/bandada. This is what it looks like : bandada.mov |
Hey @SrinivasJoshi ! Thank you so much for the idea, for implementing it and creating the video to show it. I think that your idea is a good alternative to solving this problem. You are working on the client app. I think that this app is fine. Here, the wallet is used to sign a message to generate the Semaphore identity. The problem is the dashboard app, it is I think that the solution will be something like this button on the top right. The difference would be that when the user is not on the right network, the button should say The dashboard app is using Let us know if you have more ideas, questions or feedback. |
Got it, thanks! I'll focus on the dashboard app issue 😅 |
I am applying to this issue via OnlyDust platform. My background and how it can be leveragedI would like to contribute to this problem |
Hey @SIDHARTH20K4! Do you want me to assign this issue to you? |
@vplasencia |
I am applying to this issue via OnlyDust platform. My background and how it can be leveragedHi I have been participating in hackathons from past 3 years and worked as full stack developer for 2 DEFI COMPANIES I have immense knowledge of how all dapps are built How I plan on tackling this issueWhen a user connect his wallet we check whether he is connected to our required network or not then he will have a request to change his network to our desired network just a simple task |
@vplasencia |
I am applying to this issue via OnlyDust platform. My background and how it can be leveragedAs a blockchain developer specializing in Solidity, I have extensive experience deploying smart contracts across multiple Ethereum-compatible networks. I’ve worked on various decentralized applications (dApps) and smart contracts for token management, voting systems, and financial applications. My background includes deep familiarity with testnets such as Goerli (now deprecated) and Sepolia, which is currently the preferred testnet. I have leveraged my Solidity and blockchain architecture to ensure that dApps run securely on the correct network. My ability to design contracts that integrate seamlessly with frontend requirements can help manage scenarios where users need to interact with the proper network, like Sepolia in this case. How I plan on tackling this issueNow that Goerli has been deprecated, I would prompt users to switch to the Sepolia testnet instead of Goerli. Here’s how I would approach the problem from a backend and Solidity perspective: |
I am applying to this issue via OnlyDust platform. My background and how it can be leveragedI am full stack blockchain developer I have build dapps using solidity and typescript. How I plan on tackling this issueCall this checkNetwork() function when your app loads or when performing transactions to ensure the user is connected to Goerli. |
I am applying to this issue via OnlyDust platform. My background and how it can be leveragedHello i want to take on this task |
Hey @SIDHARTH20K4 @Tchisom17 @Dhanraj30 @Benjtalkshow Thank you very much for the interest on this issue. I had to assign the issue to myself because it will be part of another issue because it will be necessary to update the wallet dependencies. Since updating the dependencies can cause updating many parts of the project and take a lot of time because there was a similar situation when trying to update ethers recently, I removed the Sorry about that. If you are still interested in contributing, please take a look at the good first issues here and feel free to ask for any other. I will be adding more good first issues soon. Thank you very much 🙏 |
If the user is connected to a network other than Goerli, ask the user to change network to Goerli
The text was updated successfully, but these errors were encountered: