Prepare support for didCatch when released

[bloodyowl]

facebook/react#10200

[bsommardahl]

I find myself needing didCatch right now. Where does this fall on the roadmap?

[bloodyowl]

I can put up a PR implementing it if looks ok to @chenglou & @rickyvetter

[Haroenv]

Would also be up for a PR if this is a feature we want

[bloodyowl]

should the signature be something like didCatch: ('self, 'error, 'info) => unit?

cc @rickyvetter

[rickyvetter]

Yeah that looks good! Should make things more concrete where possible.

info seems to have a pretty defined shape:
https://github.com/facebook/react/blob/55787006710bd8ef89a01d60c4a9cfa58c71035d/packages/react-reconciler/src/ReactFiberUnwindWork.js#L122-L124
https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/react/index.d.ts#L2223-L2228

I'm assuming error is a Js.Exn.t - but if it makes sense we could have our own type too.

[bloodyowl]

PR sent #247

[peterpme]

For the sake of cleaning up the repo (and given how old this issue is) I'm going to close this out.

Please re-open if this is still relevant. Thanks!