Skip to content

Conversation

ulissesferreira
Copy link
Contributor

@ulissesferreira ulissesferreira commented Nov 13, 2024

Explanation

As we aim to support more chains in the MetaMask extension we will need the exchange rates of the native tokens of these chains to properly value a user's account. Let's add Solana to the cryptocurrencies that we need to keep track of price.

References

Closes SOL-36

Changelog

@metamask/assets-controller

  • feat: add Solana to the polled exchange rates

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've highlighted breaking changes using the "BREAKING" category above as appropriate
  • I've prepared draft pull requests for clients and consumer packages to resolve any breaking changes

@ulissesferreira ulissesferreira requested review from a team as code owners November 13, 2024 11:54
@ulissesferreira ulissesferreira requested a review from a team November 13, 2024 11:54
zone-live
zone-live previously approved these changes Nov 13, 2024
Copy link
Member

@FrederikBolding FrederikBolding left a comment

Choose a reason for hiding this comment

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

Looks like this release is incomplete. The release tool should have updated the package.json and a follow up commit should be made that updated the changelogs.

Additionally, perhaps it is unnecessary to release every package?

@zone-live
Copy link
Contributor

@FrederikBolding I've re-run the release tool, and told it to just update the package we need assets-controllers, and bumped it's version to 43.1.0.

@FrederikBolding FrederikBolding changed the title Initialize Release 246.0.0 Release 246.0.0 Nov 13, 2024
Co-authored-by: Frederik Bolding <frederik.bolding@gmail.com>
@zone-live zone-live merged commit e06caee into main Nov 13, 2024
121 checks passed
@zone-live zone-live deleted the release/246.0.0 branch November 13, 2024 15:27
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.

3 participants