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

[Bug]: MATIC not required when bridging to MATIC #27068

Open
roymetamask opened this issue Sep 11, 2024 · 2 comments
Open

[Bug]: MATIC not required when bridging to MATIC #27068

roymetamask opened this issue Sep 11, 2024 · 2 comments
Assignees
Labels
area-design Design bug (previously known as papercuts - ask Hilary for more detail) regression-prod-12.0.6 Regression bug that was found in production in release 12.0.6 Sev2-normal Normal severity; minor loss of service or inconvenience. team-assets type-bug

Comments

@roymetamask
Copy link

Describe the bug

When bridging from Ethereum to Polygon, if user toggles on and off the ‘add gas on Polygon,’ user is able to confirm the bridge without adding gas to the receiving network If this is intended, it was very confusing to us.

Expected behavior

Expected that bridging would not be possible to Polygon network, without MATIC balance.

Screenshots/Recordings

https://www.figma.com/slides/xcGuWiaWlSb8naYqbeXErt/Team-Usability-Crew?node-id=1-62&t=EGZ3eAs46jEqOybs-1

Steps to reproduce

  1. Go to bridge in portfolio
  2. Set receiving network to Polygon
  3. Get a quote
  4. Once you see the banner to refuel MATIC on Polygon, toggle it on and off
  5. Submit the transaction

(note: you must not have MATIC already in Polygon for the account you're using)

Error messages or log output

No response

Detection stage

In production (default)

Version

12.0.6

Build type

None

Browser

Chrome

Operating system

MacOS

Hardware wallet

No response

Additional context

No response

Severity

It's not a NEGATIVE experience, but it doesn't make sense.

@metamaskbot metamaskbot added the regression-prod-12.0.6 Regression bug that was found in production in release 12.0.6 label Sep 11, 2024
@roymetamask roymetamask added the area-design Design bug (previously known as papercuts - ask Hilary for more detail) label Sep 11, 2024
@gauthierpetetin
Copy link
Contributor

Hi @roymetamask I have a few questions:

  • Is the issue on Extension side or on Portfolio side?
  • Did the behaviour change recently or has it potentially been there for a while?
  • Why would one need MATIC on recipient wallet to be able to receive tokens?

@gauthierpetetin gauthierpetetin added the Sev2-normal Normal severity; minor loss of service or inconvenience. label Sep 23, 2024
@roymetamask
Copy link
Author

roymetamask commented Sep 23, 2024

This is for Portfolio. If MATIC is not required, we should remove the toggle that currently appears:
Screenshot 2024-09-23 at 7 49 13 AM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-design Design bug (previously known as papercuts - ask Hilary for more detail) regression-prod-12.0.6 Regression bug that was found in production in release 12.0.6 Sev2-normal Normal severity; minor loss of service or inconvenience. team-assets type-bug
Projects
Status: To be fixed
Status: To be fixed
Development

No branches or pull requests

4 participants