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.

Sign up for GitHub

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

Jump to bottom

[Bug]: Contract Interaction and Alerts - MetaMask breaks with Cannot read properties of undefined (reading 'key') #27238

Open
seaona opened this issue Sep 18, 2024 · 0 comments
Open

[Bug]: Contract Interaction and Alerts - MetaMask breaks with Cannot read properties of undefined (reading 'key') #27238

seaona opened this issue Sep 18, 2024 · 0 comments
Assignees
@vinistevam
Labels
regression-RC-12.3.0 Regression bug that was found in release candidate (RC) for release 12.3.0 release-12.3.0 Issue or pull request that will be included in release 12.3.0 Sev2-normal Normal severity; minor loss of service or inconvenience. team-confirmations Push issues to confirmations team type-bug

Comments

@seaona
Copy link
Contributor

seaona commented Sep 18, 2024

Describe the bug

MetaMask broke with the error Cannot read properties of undefined (reading 'key') when I clicked on Alert, navigated over 2 alerts and then clicked Got it

Expected behavior

No response

Screenshots/Recordings

Screenshot from 2024-09-18 08-10-34

Steps to reproduce

It's not deterministic, so there must be a race condition somewhere. Here is what I did:

  1. Deploy an ERC721 contract from the test dapp
  2. Trigger a transaction with a nonce +1, so it's not confirmed
  3. Now trigger a Mint
  4. Click on the alerts (there are 2)
  5. Navigate to the 2nd alert
  6. Click got it --> MM broke

Error messages or log output

No response

Detection stage

During release testing

Version

12.3.0

Build type

None

Browser

Chrome

Operating system

Linux

Hardware wallet

No response

Additional context

No response

Severity

No response
@seaona seaona added type-bug Sev2-normal Normal severity; minor loss of service or inconvenience. team-confirmations Push issues to confirmations team release-12.3.0 Issue or pull request that will be included in release 12.3.0 labels Sep 18, 2024
@metamaskbot metamaskbot added the regression-RC-12.3.0 Regression bug that was found in release candidate (RC) for release 12.3.0 label Sep 18, 2024
@vinistevam vinistevam self-assigned this Sep 20, 2024
@vinistevam vinistevam mentioned this issue Sep 20, 2024
Open
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vinistevam vinistevam

Labels
regression-RC-12.3.0 Regression bug that was found in release candidate (RC) for release 12.3.0 release-12.3.0 Issue or pull request that will be included in release 12.3.0 Sev2-normal Normal severity; minor loss of service or inconvenience. team-confirmations Push issues to confirmations team type-bug
Projects
Bugs by severity
Status: To be fixed
Bugs by team
Status: To be fixed
Milestone
No milestone
Development

No branches or pull requests
3 participants
@metamaskbot @vinistevam @seaona