Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jordan+Fabo/1626 update sentry #1731

Merged
merged 11 commits into from
Dec 13, 2018
Merged

Jordan+Fabo/1626 update sentry #1731

merged 11 commits into from
Dec 13, 2018

Conversation

jbibla
Copy link
Collaborator

@jbibla jbibla commented Dec 10, 2018

Closes #1626

Description:

  • removed raven JS
  • added sentry/browser

❤️ Thank you!


  • Added entries in CHANGELOG.md with issue # and GitHub username
  • Reviewed Files changed in the github PR explorer

@jbibla jbibla changed the title Jordan/1626 update sentry WIP: Jordan/1626 update sentry Dec 10, 2018
@jbibla
Copy link
Collaborator Author

jbibla commented Dec 10, 2018

so it seems that the new sentry SDK has improved support for source maps - having a hard time understanding this documentation tbh.

screen shot 2018-12-10 at 2 39 39 pm

@faboweb
Copy link
Collaborator

faboweb commented Dec 11, 2018

That looks awesome!

if (optin) {
Sentry.init({
dsn: config.sentry_dsn,
release: `[email protected]`
Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah it's WIP but I was curious :)
If you want to forget aligning this with package.json we can do something like

import {version} from './package.json';

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

good idea. will fix.

@codecov
Copy link

codecov bot commented Dec 12, 2018

Codecov Report

Merging #1731 into develop will decrease coverage by 0.04%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           develop   #1731      +/-   ##
==========================================
- Coverage    97.74%   97.7%   -0.05%     
==========================================
  Files          102     102              
  Lines         2086    2090       +4     
  Branches        93      93              
==========================================
+ Hits          2039    2042       +3     
- Misses          38      39       +1     
  Partials         9       9
Impacted Files Coverage Δ
app/src/renderer/main.js 94.11% <ø> (ø) ⬆️
...src/renderer/components/common/PagePreferences.vue 100% <100%> (ø) ⬆️
app/src/renderer/vuex/modules/delegates.js 100% <100%> (ø) ⬆️
...p/src/renderer/vuex/modules/governance/deposits.js 100% <100%> (ø) ⬆️
app/src/renderer/vuex/modules/transactions.js 100% <100%> (ø) ⬆️
app/src/renderer/vuex/modules/connection.js 100% <100%> (ø) ⬆️
app/src/renderer/vuex/modules/user.js 100% <100%> (ø) ⬆️
app/src/main/index.js 96.34% <100%> (ø) ⬆️
app/src/renderer/vuex/modules/parameters.js 100% <100%> (ø) ⬆️
app/src/renderer/vuex/modules/governance/votes.js 100% <100%> (ø) ⬆️
... and 14 more

@codecov
Copy link

codecov bot commented Dec 12, 2018

Codecov Report

Merging #1731 into develop will decrease coverage by 0.04%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           develop   #1731      +/-   ##
==========================================
- Coverage    97.74%   97.7%   -0.05%     
==========================================
  Files          102     102              
  Lines         2086    2093       +7     
  Branches        93      93              
==========================================
+ Hits          2039    2045       +6     
- Misses          38      39       +1     
  Partials         9       9
Impacted Files Coverage Δ
app/src/renderer/main.js 94.11% <ø> (ø) ⬆️
...src/renderer/components/common/PagePreferences.vue 100% <100%> (ø) ⬆️
app/src/renderer/vuex/modules/delegates.js 100% <100%> (ø) ⬆️
...p/src/renderer/vuex/modules/governance/deposits.js 100% <100%> (ø) ⬆️
app/src/renderer/vuex/modules/transactions.js 100% <100%> (ø) ⬆️
app/src/renderer/vuex/modules/connection.js 100% <100%> (ø) ⬆️
app/src/renderer/vuex/modules/user.js 100% <100%> (ø) ⬆️
app/src/main/index.js 96.34% <100%> (ø) ⬆️
app/src/renderer/vuex/modules/parameters.js 100% <100%> (ø) ⬆️
app/src/renderer/vuex/modules/governance/votes.js 100% <100%> (ø) ⬆️
... and 10 more

@faboweb faboweb changed the title WIP: Jordan/1626 update sentry Jordan+Fabo/1626 update sentry Dec 13, 2018
fedekunze
fedekunze previously approved these changes Dec 13, 2018
Copy link
Contributor

@fedekunze fedekunze left a comment

Choose a reason for hiding this comment

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

utACK

@faboweb faboweb merged commit a783ff4 into develop Dec 13, 2018
@faboweb faboweb deleted the jordan/1626-update-sentry branch December 13, 2018 11:03
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.

Update Sentry library
4 participants