Skip to content

WIP - update controllers #4251

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
Closed
montelaidev wants to merge 28 commits into from
Closed

WIP - update controllers #4251

montelaidev wants to merge 28 commits into from

Conversation

@montelaidev
Copy link
Contributor

@montelaidev montelaidev commented May 3, 2024

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

montelaidev commented May 14, 2024

@metamaskbot publish-previews

@github-actions
Copy link
Contributor

github-actions bot commented May 14, 2024

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

montelaidev commented May 29, 2024

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.

2 participants

@montelaidev