-
-
Notifications
You must be signed in to change notification settings - Fork 183
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
Release 78.0.0 #1708
Release 78.0.0 #1708
Conversation
60a4ec6
to
ecca162
Compare
We should change the title of this PR to |
- **BREAKING**: `TokensController.addToken` now accepts a single options object | ||
``` | ||
{ | ||
address: string; | ||
symbol: string; | ||
decimals: number; | ||
name?: string; | ||
image?: string; | ||
interactingAddress?: string; | ||
networkClientId?: NetworkClientId; | ||
} | ||
``` | ||
- **CHANGED**: `TokensController.addToken` will use the chain ID value derived from state for `networkClientId` if provided | ||
- **CHANGED**: `TokensController.addTokens` now accepts an optional `networkClientId` as the last parameter | ||
- **CHANGED**: `TokensController.addTokens` will use the chain ID value derived from state for `networkClientId` if provided | ||
- **CHANGED**: `TokensController.watchAsset` options now accepts optional `networkClientId` which is used to get the ERC-20 token name if provided |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we link PRs related to these changes?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed in a4313b4
Thanks @mikesposito |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@metamask/asset-controllers
LGTM. Thank you for cutting this release 💪
#1633 has been merged now as well, which includes many breaking changes. We'll need to expand the scope of this release to include a few more things |
I'm working on getting #1633 included in this release right now |
Co-authored-by: Mark Stacey <markjstacey@gmail.com>
Co-authored-by: Mark Stacey <markjstacey@gmail.com>
82c50a5
to
4e3b3da
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just noticed that each of these peer dependency bumps is just called a dependency bump in the change entry, which might be confusing (as that wouldn't normally be breaking). I've added a suggestion to re-word each one to "peer dependency"
6f7da85
to
c96a938
Compare
See change logs --------- Co-authored-by: Alex Donesky <adonesky@gmail.com> Co-authored-by: Jiexi Luan <jiexiluan@gmail.com> Co-authored-by: Mark Stacey <markjstacey@gmail.com>
See change logs --------- Co-authored-by: Alex Donesky <adonesky@gmail.com> Co-authored-by: Jiexi Luan <jiexiluan@gmail.com> Co-authored-by: Mark Stacey <markjstacey@gmail.com>
See change logs