search

cerebral / overmind

Overmind - Frictionless state management
https://overmindjs.org
MIT License
1.58k stars 95 forks source link

Make loglevel of "missing devtools error" configurable #533

Closed geirsagberg closed 2 years ago

geirsagberg commented 3 years ago

It is a bit disturbing to get a huge red error in console when starting a dev app without devtools running. A warning seems more appropriate.

grandevel commented 3 years ago

Not really disturbing - if you don't intend to use the dev tools, just disable them using Overmind devtools: false. See https://overmindjs.org/api-1/createovermind.

Many devs suppress warnings, and if you are running multiple VS code instances with multiple Overmind debuggings going on, it would be easy to miss as a warning...

geirsagberg commented 3 years ago

I do use the dev tools, sometimes, just not all the time; and as for the disturbance, I suppose that is a matter of preference :) I could make the warning level configurable in a PR instead, would that be more suitable?

grandevel commented 3 years ago

I think making it configurable would be a great idea :)

geirsagberg commented 3 years ago

PR updated to make log level configurable, default is error as today.