-
-
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: jongsun/fix-composable-controller
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. |
Bitrise❌❌❌ Commit hash: 3ade7e5 Note
Tip
|
3ade7e5
to
9cfb8f5
Compare
Bitrise❌❌❌ Commit hash: abe850b Note
Tip
|
abe850b
to
61588b4
Compare
Bitrise🔄🔄🔄 Commit hash: 61588b4 Note
|
61588b4
to
d0523b0
Compare
Bitrise❌❌❌ Commit hash: d0523b0 Note
Tip
|
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
Bitrise❌❌❌ Commit hash: ff2ee98 Note
Tip
|
ff2ee98
to
0bd86a1
Compare
Bitrise✅✅✅ Commit hash: 0bd86a1 Note
|
…ineService/241202-fix-EngineService-types
Bitrise❌❌❌ Commit hash: 57d7919 Note
Tip
|
ReduxStore
, ReduxState
, optimize EngineService
event subscriptions, fix any
types in EngineService
EngineService
EngineService
EngineService
, Redux store
Quality Gate passedIssues Measures |
Description
ReduxStore
,ReduxState
-> Fixany
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^9.1.0
#10441Manual testing steps
Screenshots/Recordings
Before
After
Pre-merge author checklist
Pre-merge reviewer checklist