You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As a prerequisite for migrating MetaMetricsController to BaseController v2, and to support the TypeScript migration effort for the extension, we want to convert MetaMetricsController to TypeScript.
Scenario
No response
Design
No response
Technical Details
No response
Threat Modeling Framework
No response
Acceptance Criteria
All functions and methods are annotated with argument and return types.
All properties and free variables are annotated with types.
# is used for private properties and methods instead of _.
The test file is converted to TypeScript along with the implementation file.
Stakeholder review needed before the work gets merged
Engineering (needed in most cases)
Design
Product
QA (automation tests are required to pass before merging PRs but not all changes are covered by automation tests - please review if QA is needed beyond automation tests)
What is this about?
As a prerequisite for migrating
MetaMetricsController
to BaseController v2, and to support the TypeScript migration effort for the extension, we want to convertMetaMetricsController
to TypeScript.Scenario
No response
Design
No response
Technical Details
No response
Threat Modeling Framework
No response
Acceptance Criteria
#
is used for private properties and methods instead of_
.Stakeholder review needed before the work gets merged
References
metamask-controller.js
#23018The text was updated successfully, but these errors were encountered: