-
Notifications
You must be signed in to change notification settings - Fork 7
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
DeFi wallet login buton breaks the Next.js build #52
Comments
Hey, any updates on this ? |
|
Hello everybody, Here is the right place to report specific repository issues. For sure, there are many channels where the devs can ask for help, but reporting the issues directly on GitHub will help a lot also for tracking. We are trying to promptly address and prioritize each reported issue. However, we've noticed that some reports lack the necessary details, and others pertain to configuration or environment setup issues rather than problems with the repository itself. If there's been no activity following a report, it's likely that it's been assigned a lower priority or is awaiting additional information. I fully support the idea of requesting clarification or more details before assigning priority to issues. |
@vladimirrostok, |
Hey @CiprianDraghici, thanks for getting back to the thread to check if I still experience the issue! Sorry for my misunderstanding above, I thought there was another person with a similar problem who requested an update on this so I double-pinged the issue. I was getting this error only when attempted to run the The build works on my local setup in a project now and most likely it was me using the older version of Node before, on v21.10.0 it works fine! I updated my Node version after I hit this issue, documenting the minimum required version would be nice. The confusion originally came from me watching the last-merged PR and I noticed that the build here has failed too and the code was merged into the main #50 I didn't inspect the actual build problem back then, the issue was that the Vercel was set to Node v16, so the last build #50 failed when build #49 used Node v18 and it built fine so I thought there was a problem with the code in last pull-request #50. To sum up, there is no code issue in the repo and the actual problem was my misunderstanding here. Thanks to @MvXunity for clarifying the support channel! |
@CiprianDraghici I can confirm that the build failed for me on DigitalOcean and Vercel after I uncommented this DeFi login option and pushed the project, I have no issues building the app only on my local setup DigitalOcean project Do you have any problem deploying the last version to Vercel? It would be great If you could re-run the deployment for the last #50 pull request after specifying Node v18-20 in Vercel because it clearly failed with v16 and we didn't see any successful build there, once we see it's building nicely that would clear the whole situation and I'll look for problems within my setup. |
It looks like the latest Node 18 (18.19.1) throws this navigator error during the build, if I switch my local to Node v21 (21.1.0) it builds everything fine Vercel is limited to v18 and v20 is still in beta there, that's why my app can't be built there (v20 failed too) |
Node v21 actually has update to Vercel does currently support only v18 and v20 https://vercel.com/docs/functions/runtimes/node-js#node.js-version and there's no option to hardcode v21 for now If possible, please make sure this project in this repository builds with the current code in Vercel, re-running the build from #50 or making a new PR just to trigger the build is the best option as I'll be able to see the public Vercel check report from this PR too. @CiprianDraghici |
@vladimirrostok There's a pull request pending merging, which aims to verify that everything operates as intended. The PR will undergo several adjustments and enhancements. However, I cannot provide an estimated time of completion as we are awaiting releases for the 'sdk-core' and 'sdk-dapp' packages. |
@vladimirrostok please check this video Screen.Recording.2024-03-15.at.00.45.51.mov |
@CiprianDraghici Oh, thank you very much, this video is exactly what I needed to see! Yeah, I was really really confused here as my build is sensitive to the Node version for some reason and this problem only occurs when I uncomment the DeFi option in my app and then it fails both on local and cloud setups including Vercel. I will check everything in my application including the versions used in package .lock file, I will post the solution here once I fix this! There's a chance I've got some weird conflict somewhere in my app so I will clean it all up, and will also check if the |
Hey! I found the exact problem and fix for this The issue is here: I don't know why this was triggered in my app and was ignored in demo recording, but when I applied this workaround it always works fine for me both on local and cloud setups (Vercel + DigitalOcean project) I wrapped this button code in the
with
I think this should be prevented somewhere right in the library. Fix works for me and my apps deployed to the cloud nicely, thanks for your help @CiprianDraghici ! |
I always get this
when I uncomment the following component at the /unlock page
Other components work as expected, probably this is related to me missing some of wrappers at the code or this should work as expected if all other options work fine?
The text was updated successfully, but these errors were encountered: