Add Application.prototype.handleError

[javan]

Supply your own error handler function to override the default console.error. Useful for sending errors to a tracking service like Sentry.

const application = Application.start()

application.handleError = (error, message, detail) => {
  console.warn(message, detail)
  Raven.captureException(error)
}

[rwojsznis]

Welp, that caught me by surprise (obviously in production 😅 )

I would never-ever except such library to swallow errors for me - maybe I ask why throw is not the default here? (so if you have sentry it will be reported properly via window.onerror by default)

And ofc thanks for this PR @javan ! ❤️

[sstephenson]

@emq Could you open a new issue so your comment doesn’t get lost in the stream? Thank you!

[formigarafa]

It seems the instructions provided in the docs to customize the handleError doesn't work.
The controllers still references the original handleError not the substituted/assigned one.

To solve I had to do it like this:

import { Application } from "@hotwired/stimulus"

const handleErrorOriginal = Application.prototype.handleError
Application.prototype.handleError = function(error, message, detail) {
  console.log("now custom error handler is called");
  handleErrorOriginal.call(this, error, message, detail);
}

const application = Application.start()