-
Notifications
You must be signed in to change notification settings - Fork 98
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
Conversation
That looks awesome! |
app/src/main/index.js
Outdated
if (optin) { | ||
Sentry.init({ | ||
dsn: config.sentry_dsn, | ||
release: `[email protected]` |
There was a problem hiding this comment.
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';
There was a problem hiding this comment.
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 Report
@@ 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
|
Codecov Report
@@ 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
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utACK
Closes #1626
Description:
❤️ Thank you!
CHANGELOG.md
with issue # and GitHub usernameFiles changed
in the github PR explorer