-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
bump @metamask/smart-transactions-controller
version
#17120
Conversation
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
4834318
to
8fbf098
Compare
Builds ready [8fbf098]
Page Load Metrics (1558 ± 263 ms)
Bundle size diffs [🚀 Bundle size reduced!]
|
@metamask/smart-transactions-controller
version
this.networkController.store.subscribe((networkState) => { | ||
const modifiedNetworkState = { | ||
...networkState, | ||
providerConfig: { |
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.
the smart-transaction-controller
types the networkState using a type from @metamask/network-controller
(from @metamask/core-monorepo
) that was recently updated to expect property providerConfig
in place of provider
: https://github.com/MetaMask/core/blob/main/packages/network-controller/CHANGELOG.md?plain=1#L11
Explanation
Bumps
@metamask/smart-transactions-controller
version tov3.1.0
- a version which replace use of full@metamask/controllers
repo with packages from@metamask/core-monorepo
. This change allows us to reduce our unused code burden since@metamask/controllers
was bringing lots of code we don't currently use into our bundle (the controllers version ofKeyringController
, with many additional dependencies, as one example).Bundle size reduced by .8MB :
This change should entail no functional changes.