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

Switching Pilot account is not working well in Meta Mask #422

Closed
jfschwarz opened this issue Dec 23, 2024 · 3 comments
Closed

Switching Pilot account is not working well in Meta Mask #422

jfschwarz opened this issue Dec 23, 2024 · 3 comments
Assignees

Comments

@jfschwarz
Copy link
Collaborator

jfschwarz commented Dec 23, 2024

When switching the connected address in Meta Mask this should be reflected in Pilot, but users report that they are unable to connect Pilot to the desired account. see this internal discord chat, for example: https://discord.com/channels/881881751369175040/1169364104373354586/1320714892482711562

Steps to reproduce:

Note:
If https://connect.pilot.gnosisguild.org/ isn't connected to any MM account, it will actually present a connection confirmation modal and you are able to connect the desired Pilot address.

@jfschwarz jfschwarz added the bug label Dec 23, 2024
@jfschwarz jfschwarz changed the title Switching Pilot addr Switching Pilot account is not working well Dec 23, 2024
@frontendphil frontendphil self-assigned this Jan 6, 2025
@jfschwarz jfschwarz changed the title Switching Pilot account is not working well Switching Pilot account is not working well in Meta Mask Jan 6, 2025
@frontendphil
Copy link
Member

Managing routes and connections through the extension might make this harder than it needs to be. @jfschwarz has an idea to extract the route configuration UI into the connect.pilot.gnosisguild.org domain. A small app that contains the current behavior. This would have the advantage that we wouldn't need to manage the MetaMask connection through an iframe, but it could be connected to the same host.

@frontendphil
Copy link
Member

Postponing a fix in the current app version in favor of implementing #469 should also eliminate this bug. I'm keeping it open for visibility, though.

@frontendphil frontendphil removed the bug label Jan 22, 2025
@frontendphil
Copy link
Member

I tested this today, and it is fixed with the latest changes to the new routes config in the companion app.

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

No branches or pull requests

2 participants