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 AppStateController to BaseController v2, and to support the TypeScript migration effort for the extension, we want to convert AppStateController to TypeScript along with its tests.
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.
Relevant types from app-state.d.ts are folded into app-state.ts.
# 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
AppStateController
to BaseController v2, and to support the TypeScript migration effort for the extension, we want to convertAppStateController
to TypeScript along with its tests.Scenario
No response
Design
No response
Technical Details
No response
Threat Modeling Framework
No response
Acceptance Criteria
app-state.d.ts
are folded intoapp-state.ts
.#
is used for private properties and methods instead of_
.Stakeholder review needed before the work gets merged
References
metamask-controller.js
#23017The text was updated successfully, but these errors were encountered: