-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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.
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
fix: Type fixes and optimizations for EngineService
, Redux store
#12509
base: main
Are you sure you want to change the base?
fix: Type fixes and optimizations for EngineService
, Redux store
#12509
Conversation
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
This comment was marked as outdated.
This comment was marked as outdated.
3ade7e5
to
9cfb8f5
Compare
This comment was marked as outdated.
This comment was marked as outdated.
abe850b
to
61588b4
Compare
This comment was marked as duplicate.
This comment was marked as duplicate.
61588b4
to
d0523b0
Compare
This comment was marked as outdated.
This comment was marked as outdated.
EngineService
typesEngineService
, Redux store
d0523b0
to
fa4ea81
Compare
d484693
to
d28dd1f
Compare
fa4ea81
to
ff2ee98
Compare
EngineService
, Redux storeReduxStore
, ReduxState
, and fix any
types in EngineService
ReduxStore
, ReduxState
, and fix any
types in EngineService
ReduxStore
, ReduxState
, optimize EngineService
event subscriptions, fix any
types in EngineService
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice, LGTM
3d616b0
to
1f5952e
Compare
Bitrise✅✅✅ Commit hash: 1f5952e Note
|
… replace with new types in `app/core/redux`
Bitrise✅✅✅ Commit hash: c2e4ecb Note
|
Quality Gate passedIssues Measures |
Description
Define types:(superseded by chore: Chore/12435 mvp handle engine does not exist #12538)ReduxStore
,ReduxState
any
types inEngineService
.EngineService
event subscriptions:update_bg_state_cb
callback being re-defined on every iteration.stateChange
events collections intoBACKGROUND_STATE_CHANGE_EVENT_NAMES
constant for maintainability.Related issues
@metamask/composable-controller
from^3.0.0
to^10.0.0
#10441Manual testing steps
Screenshots/Recordings
Pre-merge author checklist
Pre-merge reviewer checklist