-
Notifications
You must be signed in to change notification settings - Fork 55
Severity #21
Conversation
@@ -21,6 +23,13 @@ module.exports = class Notification | |||
event.appVersion = Configuration.appVersion if Configuration.appVersion | |||
event.releaseStage = Configuration.releaseStage if Configuration.releaseStage | |||
|
|||
event.payloadVersion = Configuration.payloadVersion if Configuration.payloadVersion | |||
|
|||
if options.severity? and SUPPORTED_SEVERITIES.include?(options.severity) |
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.
Does .include? work — I though that was a ruby thing.
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.
No, it doesn't, and I had the right thing there at first too. Was switching between this and Ruby, must have gotten twisted up :)
@@ -125,6 +125,7 @@ module.exports = Bugsnag = (function() { | |||
} | |||
dom = domain.create(); | |||
dom._bugsnagOptions = options; | |||
options["severity"] = "error"; |
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.
I think we also need to do this in requestHandler
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.
👍
Do we want severity=error on |
We definitely do want it in |
Other than that, 🍬 |
Add severities & payloadVersion.
Related Pull Requests:
bugsnag/bugsnag-website#189
bugsnag/bugsnag-notifier-api#3
bugsnag/bugsnag-event-worker#59
bugsnag/bugsnag-ruby#96
bugsnag/bugsnag-php#39
bugsnag/bugsnag-cocoa#14
bugsnag/bugsnag-java#13
bugsnag/bugsnag-js#41
bugsnag/bugsnag-android#15
bugsnag/bugsnag-python#23