metamask-extension icon indicating copy to clipboard operation
metamask-extension copied to clipboard

[Bug]: After adding a custom network I am unable to see the tokens in the imported wallet

Open davibroc opened this issue 1 year ago • 2 comments

Describe the bug

It used be able to add Tenderly as a customer network and import the tokens in the account. But that no longer works and because of that I am able to test the extension using Tenderly and to merge my e2e tests. This functionality was broken by the following PR https://github.com/MetaMask/metamask-extension/pull/22814 See the screen recording that shows the problem before and after the PR was merged

Expected behavior

I am able to import the tokens in the wallet when using a custom network

Screenshots/Recordings

Before:

https://github.com/MetaMask/metamask-extension/assets/6082863/ac627c6b-c68b-4c78-9866-7aa455e97a1f

After:

https://github.com/MetaMask/metamask-extension/assets/6082863/7b62026d-0453-40db-aa3e-1c771967042d

Steps to reproduce

  1. Create a new account in the extension
  2. Import a wallet using this seed phrase buzz pill embody elite name festival crystal pigeon grief memory allow blue
  3. After the account is imported add a custom network: name: 'Tenderly', url: 'https://rpc.tenderly.co/fork/c2e7fb02-9c2e-40ee-b911-ebf5b0326f96', chainID: '1', symbol: 'ETH',
  4. After switching the network to Tenderly click on the Refresh list button
  5. Notice that the tokens in the account cannot be imported

Error messages or log output

No response

Version

11.8.0

Build type

None

Browser

Chrome

Operating system

MacOS

Hardware wallet

No response

Additional context

No response

Severity

No response

davibroc avatar Feb 16 '24 09:02 davibroc

@davibroc this has a label "regression-prod-11.8.0". Does that mean you can reproduce this in production? Or only on our develop branch?

danjm avatar Feb 16 '24 20:02 danjm

Since the linked PR was checked in a few days ago this should not be in 11.8 or any release yet. I did attempt a fix here, which worked for me locally and I think makes sense: https://github.com/MetaMask/core/pull/3927

bergeron avatar Feb 16 '24 23:02 bergeron

@davibroc this has a label "regression-prod-11.8.0". Does that mean you can reproduce this in production? Or only on our develop branch?

Yes my fault. Only happens on develop. I have removed it

davibroc avatar Feb 20 '24 06:02 davibroc

This issue has been automatically marked as stale because it has not had recent activity in the last 90 days. It will be closed in 45 days if there is no further activity. The MetaMask team intends on reviewing this issue before close, and removing the stale label if it is still a bug. We welcome new comments on this issue. We do not intend on closing issues if they report bugs that are still reproducible. Thank you for your contributions.

github-actions[bot] avatar May 20 '24 22:05 github-actions[bot]

This issue was closed because there has been no follow up activity in the last 45 days. If you feel this was closed in error, please reopen and provide evidence on the latest release of the extension. Thank you for your contributions.

github-actions[bot] avatar Jul 04 '24 22:07 github-actions[bot]