Skip to content

Enable token detection for the Aurora network#1327

Merged
dan437 merged 4 commits intoMetaMask:mainfrom
dan437:token-detection-aurora
May 8, 2023
Merged

Enable token detection for the Aurora network#1327
dan437 merged 4 commits intoMetaMask:mainfrom
dan437:token-detection-aurora

Conversation

@dan437
Copy link
Copy Markdown
Contributor

@dan437 dan437 commented May 4, 2023

Description

Enable token detection for the Aurora network.

Copy link
Copy Markdown
Contributor

@mcmire mcmire left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see that we are making it so that isTokenDetectionSupportedForNetwork returns true for Aurora, but this assumes that we're using that function somewhere. Is there a way to prove that token detection works for Aurora? Perhaps we could copy this test: we could use Nock to mock the token API for Aurora so that it returns some tokens, configures the controller for Aurora, calls start, and confirms that Aurora is being hit and returns a set of tokens.

Copy link
Copy Markdown
Contributor

@mcmire mcmire left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@dan437 dan437 merged commit 614fb7b into MetaMask:main May 8, 2023
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

Successfully merging this pull request may close these issues.

2 participants