Skip to content
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.

Sign up for GitHub

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

Jump to bottom

WIP - update controllers #4251

Closed
wants to merge 28 commits into from
Closed

WIP - update controllers #4251

wants to merge 28 commits into from

Conversation

montelaidev
Copy link
Contributor

Explanation

References

Changelog

@metamask/package-a

  • : Your change here
  • : Your change here

@metamask/package-b

  • : Your change here
  • : Your change here

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

@montelaidev
feat: add mocks and evm check to accounts controller
0eff976
@montelaidev
feat: add mocks and util function for non evm
83250ef
@montelaidev
fix: update NftDetectionController to use selectedInternalId instead …
fafd322 
…of selectedAddress
@montelaidev
fix: update NftController to use selectedAccountId instead of selecte…
61ace68 
…dAddress
@montelaidev
fix: use selectedAccount action
c866813
@montelaidev
fix: listeners in NftDetectionController
6ee9a62
@montelaidev
feat: add mocks and evm check to accounts controller
4c30c69
@montelaidev
feat: add mocks and util function for non evm
c0cde1b
@montelaidev
fix: remove getIdentites and getSelectedAddress from AccountTrackerCo…
a9ec2a0 
…ntrolelr and use InternalAccount
@montelaidev
fix: update selectedAddress args in token controllers
23aeaa0
@montelaidev
fix: udpate TokenBalancesController to use InternalAccount instead of…
42133c0 
… selectedAddress
@montelaidev
fix: update TokenBalances test to reach 100% coverage
e291bdb
@montelaidev
fix: update token controllers
9b8a1eb
@montelaidev
feat: add tests to util and mocks
6c0595d
@montelaidev
fix: remove nonevm check during selectedAccountChange
cb17e99
@montelaidev
feat: add mocks and evm check to accounts controller
f5f6c11
@montelaidev
feat: add mocks and util function for non evm
5967095
@montelaidev
fix: imports
0e5df33
@montelaidev
Merge branch 'fix/ap381/update-account-tracker-to-use-selectedAccount…
9dd7c8d 
…Id' into fix/ap381/update-controllers-to-use-selectedAccountId
@montelaidev
Merge branch 'fix/ap381/update-tokens-controllers-to-use-internal-acc…
1c9e453 
…ounts' into fix/ap381/update-controllers-to-use-selectedAccountId
@montelaidev
Merge branch 'fix/ap381/update-nft-controllers-to-use-internal-accoun…
0b8cbed 
…t' into fix/ap381/update-controllers-to-use-selectedAccountId
@montelaidev
Merge branch 'fix/ap381/update-transaction-controllers-to-use-selecte…
6e656f6 
…dAccountId' into fix/ap381/update-controllers-to-use-selectedAccountId
@montelaidev
fix: peer dep version
a7089c5
@montelaidev
Merge remote-tracking branch 'origin/main' into fix/ap381/update-cont…
cff6cd9 
…rollers-to-use-selectedAccountId
@montelaidev
fix: update to keyring-api 6.1
8cee6a6
@montelaidev
chore(deps): bump keyring api version to 6.1.0
eb5191e
@montelaidev
fix: type error
f916816
@montelaidev
temp: add skip lib check for snaps sdk type error
a1db071
@montelaidev
Copy link
Contributor Author

@metamaskbot publish-previews

@github-actions GitHub Actions
Copy link
Contributor

Preview builds have been published. See these instructions for more information about preview builds.

Expand for full list of packages and versions. 
{
  "@metamask-previews/accounts-controller": "14.0.0-preview-a1db0714",
  "@metamask-previews/address-book-controller": "4.0.1-preview-a1db0714",
  "@metamask-previews/announcement-controller": "6.1.0-preview-a1db0714",
  "@metamask-previews/approval-controller": "6.0.2-preview-a1db0714",
  "@metamask-previews/assets-controllers": "29.0.0-preview-a1db0714",
  "@metamask-previews/base-controller": "5.0.2-preview-a1db0714",
  "@metamask-previews/build-utils": "2.0.1-preview-a1db0714",
  "@metamask-previews/composable-controller": "6.0.1-preview-a1db0714",
  "@metamask-previews/controller-utils": "9.1.0-preview-a1db0714",
  "@metamask-previews/ens-controller": "10.0.1-preview-a1db0714",
  "@metamask-previews/eth-json-rpc-provider": "3.0.2-preview-a1db0714",
  "@metamask-previews/gas-fee-controller": "15.1.1-preview-a1db0714",
  "@metamask-previews/json-rpc-engine": "8.0.2-preview-a1db0714",
  "@metamask-previews/json-rpc-middleware-stream": "7.0.1-preview-a1db0714",
  "@metamask-previews/keyring-controller": "16.0.0-preview-a1db0714",
  "@metamask-previews/logging-controller": "3.0.1-preview-a1db0714",
  "@metamask-previews/message-manager": "8.0.2-preview-a1db0714",
  "@metamask-previews/name-controller": "6.0.1-preview-a1db0714",
  "@metamask-previews/network-controller": "18.1.0-preview-a1db0714",
  "@metamask-previews/notification-controller": "5.0.1-preview-a1db0714",
  "@metamask-previews/permission-controller": "9.0.2-preview-a1db0714",
  "@metamask-previews/permission-log-controller": "2.0.1-preview-a1db0714",
  "@metamask-previews/phishing-controller": "9.0.2-preview-a1db0714",
  "@metamask-previews/polling-controller": "6.0.2-preview-a1db0714",
  "@metamask-previews/preferences-controller": "11.0.0-preview-a1db0714",
  "@metamask-previews/queued-request-controller": "0.9.0-preview-a1db0714",
  "@metamask-previews/rate-limit-controller": "5.0.1-preview-a1db0714",
  "@metamask-previews/selected-network-controller": "12.0.1-preview-a1db0714",
  "@metamask-previews/signature-controller": "16.0.0-preview-a1db0714",
  "@metamask-previews/transaction-controller": "28.1.1-preview-a1db0714",
  "@metamask-previews/user-operation-controller": "9.0.0-preview-a1db0714"
}

@montelaidev
Copy link
Contributor Author

Closed for #4291

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@montelaidev