Skip to content

Conversation

@vdh
Copy link
Contributor

@vdh vdh commented Oct 21, 2016

I noticed when I enabled diffs that I was getting [object Object] in the diffs. I made this change so that instead of stringifying the output from deep-diff, the values are sent directly to logger.log.

I don't know about console implementations outside of Chrome, but Chrome's console works great at displaying the objects in a nice format for inspection.

@codecov-io
Copy link

Current coverage is 9.32% (diff: 0.00%)

Merging #182 into master will not change coverage

@@            master      #182   diff @@
========================================
  Files            5         5          
  Lines          118       118          
  Methods          0         0          
  Messages         0         0          
  Branches         0         0          
========================================
  Hits            11        11          
  Misses         107       107          
  Partials         0         0          

Powered by Codecov. Last update 781f790...9ba744f

@grushetsky grushetsky self-assigned this Oct 21, 2016
@sibelius
Copy link
Contributor

sibelius commented Jan 9, 2017

@grushetsky looks good to me

@imevro imevro merged commit 8c916d0 into LogRocket:master Feb 1, 2017
@imevro
Copy link
Collaborator

imevro commented Feb 1, 2017

Thanks!

@vdh vdh deleted the log-diff-values-directly branch February 1, 2017 23:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants