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]: Unable to save advanced gas settings when interacting with test dapp contracts - SAVE button greyed out #27608

Closed
Unik0rnMaggie opened this issue Oct 3, 2024 · 5 comments
Labels
regression-prod-* Regression bug that was found in production in release * Sev2-normal Normal severity; minor loss of service or inconvenience. team-confirmations Push issues to confirmations team type-bug

Comments

@Unik0rnMaggie
Copy link
Contributor

Describe the bug

When trying to interact with failing and multisig test dapp contracts, the SAVE button on the advanced gas settings screen is greyed out.

Expected behavior

Should be able to click the SAVE button on the advanced gas settings screen.

Screenshots/Recordings

Screen.Recording.2024-10-03.at.21.25.04.mov
Screen.Recording.2024-10-03.at.21.27.56.mov

Steps to reproduce

  1. Connect to the test dapp
  2. Deploy a failing contract or a multisig contract
  3. Send a failing transaction or send to multisig address
  4. Edit the gas fees and select Advanced
  5. See that the SAVE button is greyed out

Error messages or log output

No response

Detection stage

During release testing

Version

12.4.0

Build type

Other (please specify exactly where you obtained this build in "Additional Context" section)

Browser

Chrome, Firefox

Operating system

MacOS

Hardware wallet

No response

Additional context

I obtained the build from here

Severity

No response

@github-project-automation github-project-automation bot moved this to To be fixed in Bugs by team Oct 3, 2024
@github-project-automation github-project-automation bot moved this to To be fixed in Bugs by severity Oct 3, 2024
@benjisclowder benjisclowder added the Sev2-normal Normal severity; minor loss of service or inconvenience. label Oct 8, 2024
@bschorchit
Copy link

bschorchit commented Oct 8, 2024

I was not able to repro this issue for send to a multisig on a v12.4 build:
https://github.com/user-attachments/assets/d41e0f55-1410-4628-a9ab-18bc61676e08

@bschorchit
Copy link

I was able to reproduce it for send failing transaction on both v12.4 and v12.3.1

@bschorchit bschorchit added regression-prod-* Regression bug that was found in production in release * and removed regression-RC-12.4.0 labels Oct 8, 2024
@bschorchit
Copy link

I think the issue is due to the gas limit being higher than our limits.
Screenshot 2024-10-08 at 14 00 22

@bschorchit
Copy link

Related with #27089 that was fixed on with #27165

@bschorchit
Copy link

I have tested now with the build from this PR #27165 and noticed the issue is fixed so closing this issue.

@github-project-automation github-project-automation bot moved this from To be fixed to Fixed in Bugs by team Oct 8, 2024
@github-project-automation github-project-automation bot moved this from To be fixed to Fixed in Bugs by severity Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
regression-prod-* Regression bug that was found in production in release * Sev2-normal Normal severity; minor loss of service or inconvenience. team-confirmations Push issues to confirmations team type-bug
Projects
Archived in project
Development

No branches or pull requests

3 participants