Skip to content
This repository has been archived by the owner on Jun 4, 2024. It is now read-only.

Remove redux-logger? #117

Closed
chriddyp opened this issue Feb 6, 2019 · 1 comment
Closed

Remove redux-logger? #117

chriddyp opened this issue Feb 6, 2019 · 1 comment
Assignees

Comments

@chriddyp
Copy link
Member

chriddyp commented Feb 6, 2019

The redux logger causes a ton of output in the console, especially with hot loading.

Also, sometimes it catches component error messages and rethrows them, making the tracebacks & "launch debugger on uncaught exceptions" tools hard to use.

I'm in favor of removing redux-logger. If anyone benefits from this while developing dash-renderer, they can add it back while they are in development (but not commit it).

Here's an associated community report that is having trouble filtering out the logs: https://community.plot.ly/t/dash-renderer-dev-js-logs-messes-my-browser-console/19120

@T4rk1n
Copy link
Contributor

T4rk1n commented Feb 6, 2019

Agreed, also I added redux dev tools, it can be used to see the store update instead. #107

@T4rk1n T4rk1n self-assigned this Feb 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants