Skip to content

fix(browser): do not override ls debug if found#331

Merged
thebigredgeek merged 1 commit intodebug-js:masterfrom
levithomason:patch-1
Nov 14, 2016
Merged

fix(browser): do not override ls debug if found#331
thebigredgeek merged 1 commit intodebug-js:masterfrom
levithomason:patch-1

Conversation

@levithomason
Copy link
Copy Markdown
Contributor

@levithomason levithomason commented Nov 13, 2016

In React apps built with Webpack process.env.NODE_ENV is set as a global. This means process.env is a global object. The current browser code overrides whatever is found in localStorage. This PR returns early if a debug value is found in localStorage.

Fixes #330

@TooTallNate
Copy link
Copy Markdown
Contributor

LGTM /cc @thebigredgeek @ibc

Fixes #330 I think.

@levithomason
Copy link
Copy Markdown
Contributor Author

levithomason commented Nov 13, 2016

I believe so, exactly my issue. I've closed the issue via the description 👍

@ibc
Copy link
Copy Markdown
Contributor

ibc commented Nov 13, 2016

It seems that 20c37fd should fix #330 issue. Hope a new NPM version 2.3.3 is released soon.

@thebigredgeek
Copy link
Copy Markdown
Contributor

Yeah I'll release this morning once I get to office :)

@thebigredgeek thebigredgeek added bug This issue identifies a malfunction change-patch This proposes or provides a change that requires a patch release labels Nov 14, 2016
@thebigredgeek thebigredgeek merged commit 2caf4ef into debug-js:master Nov 14, 2016
@levithomason levithomason deleted the patch-1 branch November 14, 2016 17:26
@Ambroos
Copy link
Copy Markdown

Ambroos commented Nov 17, 2016

@thebigredgeek Any news? 2.3.2 is broken in quite a few cases and a release would be very welcome.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug This issue identifies a malfunction change-patch This proposes or provides a change that requires a patch release

Development

Successfully merging this pull request may close these issues.

5 participants