Skip to content

Commit 7646ea4

Browse files
perf(overmind-devtools-client): fix production build and only show console in development
1 parent 26226bf commit 7646ea4

File tree

3 files changed

+19
-15
lines changed

3 files changed

+19
-15
lines changed

packages/node_modules/overmind-devtools-client/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
"repository": "git+https://github.com/cerebral/overmind.git",
88
"scripts": {
99
"start": "webpack-dev-server",
10-
"build": "webpack",
10+
"build": "webpack --mode production",
1111
"typecheck": "tsc --noEmit",
1212
"test": "jest --runInBand",
1313
"test:watch": "jest --watch --updateSnapshot --coverage false",

packages/node_modules/overmind-devtools-client/src/components/Tabs/index.tsx

Lines changed: 13 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -100,17 +100,19 @@ const Tabs: FunctionComponent = () => {
100100
<FaSave />
101101
</button>
102102
</Tooltip>
103-
<Tooltip text="Console">
104-
<button
105-
className={css(
106-
styles.button,
107-
state.currentTab === Tab.Console && styles.activeButton
108-
)}
109-
onClick={() => actions.changeTab(Tab.Console)}
110-
>
111-
<FaTerminal />
112-
</button>
113-
</Tooltip>
103+
{process.env.NODE_ENV === 'development' ? (
104+
<Tooltip text="Console">
105+
<button
106+
className={css(
107+
styles.button,
108+
state.currentTab === Tab.Console && styles.activeButton
109+
)}
110+
onClick={() => actions.changeTab(Tab.Console)}
111+
>
112+
<FaTerminal />
113+
</button>
114+
</Tooltip>
115+
) : null}
114116
</div>
115117
)
116118
}

packages/node_modules/overmind-devtools-client/src/overmind/operators.ts

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -98,9 +98,11 @@ export const ensureApp: () => Operator<Message> = () =>
9898

9999
export const addClientMessage: () => Operator<Message> = () =>
100100
mutate(({ state }, message) => {
101-
state.apps[message.appName].messages.unshift(
102-
JSON.parse(JSON.stringify(message.message))
103-
)
101+
if (process.env.NODE_ENV === 'development') {
102+
state.apps[message.appName].messages.unshift(
103+
JSON.parse(JSON.stringify(message.message))
104+
)
105+
}
104106
})
105107

106108
export const addComponent: () => Operator<AddComponentMessage> = () =>

0 commit comments

Comments
 (0)