Cawllec/fix shouldnt notify logging

[Cawllec]

Disables delivery errors in the case that shouldNotify is false, adds configuration option to overwrite this behaviour if necessary

[Cawllec]

Based on discussion with @bengourley I've removed the changes based around the configuration options, and instead ensured that when an error occurs and is correctly not-notified the original error is logged instead of the notification error.

This still leaves the issue with the initial log level however.

[bengourley]

This is better!

Here is how the user's repro repo now behaves:

I think it would be good if the error stack got into the log and looked more like how uncaught errors get logged out:

appending e.stack to the log message should do the trick?

[Cawllec]

Appended that stack as suggested and removed the Bugsnag: repetition.

I notice that it's logging each issue twice, any idea why?

[bengourley]

LGTM 👍