Skip to content

Commit

Permalink
fix: DevToolsManager() config parameter
Browse files Browse the repository at this point in the history
  • Loading branch information
ntucker committed May 8, 2024
1 parent 30208fb commit c3481ad
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
5 changes: 5 additions & 0 deletions .changeset/tall-cheetahs-cheat.md
@@ -0,0 +1,5 @@
---
"@data-client/core": patch
---

Fix DevToolsManager() config parameter correctly sets devtools config
4 changes: 2 additions & 2 deletions packages/core/src/manager/DevtoolsManager.ts
Expand Up @@ -94,7 +94,7 @@ export default class DevToolsManager implements Manager {
(window as any).__REDUX_DEVTOOLS_EXTENSION__ &&
(window as any).__REDUX_DEVTOOLS_EXTENSION__.connect({
...DEFAULT_CONFIG,
config,
...config,
});
if (process.env.NODE_ENV !== 'production' && this.devTools) {
this.devTools.subscribe((msg: any) => {
Expand Down Expand Up @@ -150,7 +150,7 @@ export default class DevToolsManager implements Manager {

handleAction(action: any, state: any) {
if (this.started) {
this.devTools.send(action, state, undefined, 'RDC');
this.devTools.send(action, state);
} else {
// avoid this getting too big in case this is long running
// we cut in half so we aren't constantly reallocating
Expand Down

0 comments on commit c3481ad

Please sign in to comment.