Skip to content

Feat: Send console logs to Sentry#63

Open
bhawesh96 wants to merge 1 commit into
masterfrom
sentry
Open

Feat: Send console logs to Sentry#63
bhawesh96 wants to merge 1 commit into
masterfrom
sentry

Conversation

@bhawesh96

Copy link
Copy Markdown
Member

Description

Using Raven for sending console errors on sentry dashboard.
Requires specific npm, node versions and hence updated engines in package.json

Type of Change:

  • New feature (non-breaking change which adds functionality)

Comment thread src/index.js
linesOfContext: 40
}).install()

var originalConsoleError = console.error

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use const instead of var to be ES6 consistent

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please pull and do it off. I'm unable to fetch time.

@rsarky rsarky force-pushed the master branch 2 times, most recently from e9a1a87 to 1e6c233 Compare January 9, 2019 15:37
@rsarky

rsarky commented Jan 9, 2019

Copy link
Copy Markdown
Member

Ping

@rsarky rsarky force-pushed the master branch 2 times, most recently from 66c278c to ee9a445 Compare January 13, 2019 17:22
Using Raven for sending console errors on sentry dashboard
Requires specific npm, node versions
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.

2 participants