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

hotfix: bump eth-connect dependency version #6005

Closed
wants to merge 3 commits into from

Conversation

meelrossi
Copy link
Contributor

@meelrossi meelrossi commented Dec 5, 2023

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 valid

How 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 the browser-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.

@sandrade-dcl sandrade-dcl changed the base branch from dev to main December 5, 2023 14:33
@sandrade-dcl sandrade-dcl changed the title fix: bump eth-connect dependency version hotfix: bump eth-connect dependency version Dec 5, 2023
Copy link
Contributor

@LucasLioyQA LucasLioyQA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA approved on .net and .org using logging in with both: guest and wallet:

Screenshot 2023-12-05 at 11 46 27 Screenshot 2023-12-05 at 11 47 44 Screenshot 2023-12-05 at 11 49 04 Screenshot 2023-12-05 at 11 49 37

@Ludmilafantaniella Ludmilafantaniella self-requested a review December 5, 2023 20:43
Copy link
Contributor

@Ludmilafantaniella Ludmilafantaniella left a 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.

@sandrade-dcl sandrade-dcl deleted the fix/bump-eth-connect-version branch December 6, 2023 12:47
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.

6 participants