core
core copied to clipboard
chore: adds new initialize method and cleans constructor
Explanation
We are removing the async call that existed in the constructor and moving it to a initialize method, the goal is to not have side-effects such as this call inside a constructor.
References
Ref slack conversation. https://github.com/MetaMask/accounts-planning/issues/927
Changelog
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 communicated my changes to consumers by updating changelogs for packages I've changed, highlighting breaking changes as necessary
- [ ] I've prepared draft pull requests for clients and consumer packages to resolve any breaking changes