Skip to content

Commit

Permalink
Merge pull request #5022 from MetaMask/sentry-release-version
Browse files Browse the repository at this point in the history
sentry - change releaseVersion to release
  • Loading branch information
jennypollack authored Aug 9, 2018
2 parents 2b25902 + 00a2ab2 commit 0442e5d
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions app/scripts/background.js
Original file line number Diff line number Diff line change
Expand Up @@ -44,8 +44,8 @@ const notificationManager = new NotificationManager()
global.METAMASK_NOTIFIER = notificationManager

// setup sentry error reporting
const releaseVersion = platform.getVersion()
const raven = setupRaven({ releaseVersion })
const release = platform.getVersion()
const raven = setupRaven({ release })

// browser check if it is Edge - https://stackoverflow.com/questions/9847580/how-to-detect-safari-chrome-ie-firefox-and-opera-browser
// Internet Explorer 6-11
Expand Down
4 changes: 2 additions & 2 deletions app/scripts/lib/setupRaven.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ module.exports = setupRaven

// Setup raven / sentry remote error reporting
function setupRaven (opts) {
const { releaseVersion } = opts
const { release } = opts
let ravenTarget
// detect brave
const isBrave = Boolean(window.chrome.ipcRenderer)
Expand All @@ -22,7 +22,7 @@ function setupRaven (opts) {
}

const client = Raven.config(ravenTarget, {
releaseVersion,
release,
transport: function (opts) {
opts.data.extra.isBrave = isBrave
const report = opts.data
Expand Down

0 comments on commit 0442e5d

Please sign in to comment.