-
Notifications
You must be signed in to change notification settings - Fork 93
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
hotfix: bump eth-connect dependency version #6005
Conversation
chore: sync main to dev
show back pending status warning for email subscriptions
After the CI passes: WebThis branch can be previewed at:
Desktop:If you have the launcher installed (download launcher) you can press open on the following link: SDK 6/7:More |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.../Assets/Scripts/MainScripts/DCL/Controllers/HUD/MyAccountHUD/EmailNotificationsController.cs
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅ Review completed and fix verified. Firefox login, as guest and connecting a wallet, works as expected.
What does this PR change?
Bump
eth-connect
version to include fix that was causing loging in in firefox not to work. We are checking that the error key is not present or is undefined to make a response validHow to test the changes?
Log in using firefox to https://play.decentraland.org/?explorer-branch=fix/bump-eth-connect-version
Our Code Review Standards
https://github.com/decentraland/unity-renderer/blob/master/docs/code-review-standards.md
Copilot summary
🤖[deprecated] Generated by Copilot at a5107af
This pull request updates the
eth-connect
dependency in thebrowser-interface
package to the latest version. This allows the package to use the latest features and bug fixes of the library for interacting with Ethereum smart contracts.