Merge pull request #53 from rescript-lang/fix-error-boundary

Fix ErrorBoundary component using external binding

Author: cristianoc

.gitignore

@@ -17,3 +17,6 @@ _build
 
 # Editor
 /.idea/
+
+# React
+!/src/react/*.js

src/RescriptReactErrorBoundary.res

@@ -3,34 +3,36 @@
  * As soon as React provides a mechanism for error-catching using functional component,
  * this is likely to be deprecated and/or move to user space.
  ")
-type reactComponentClass
-
-@module("react") external component: reactComponentClass = "Component"
-
 type info = {componentStack: string}
 
 type params<'error> = {
   error: 'error,
   info: info,
 }
 
-let getErrorBoundary: reactComponentClass => React.element = %raw(`
-  function (Component) {
-    function ErrorBoundary(props) {
-      Component.call(this);
-      this.state = {error: undefined};
-    };
-    ErrorBoundary.prototype = Object.create(Component.prototype);
-    ErrorBoundary.prototype.componentDidCatch = function(error, info) {
-      this.setState({error: {error: error, info: info}})
-    };
-    ErrorBoundary.prototype.render = function() {
-      return this.state.error != undefined ? this.props.fallback(this.state.error) : this.props.children
-    }
-    return ErrorBoundary;
+%%raw(`
+var React = require("react");
+
+var ErrorBoundary = (function (Component) {
+  function ErrorBoundary(props) {
+    Component.call(this);
+    this.state = { error: undefined };
   }
+  ErrorBoundary.prototype = Object.create(Component.prototype);
+  ErrorBoundary.prototype.componentDidCatch = function (error, info) {
+    this.setState({ error: { error: error, info: info } });
+  };
+  ErrorBoundary.prototype.render = function () {
+    return this.state.error != undefined
+      ? this.props.fallback(this.state.error)
+      : this.props.children;
+  };
+  return ErrorBoundary;
+})(React.Component);
 `)
 
-@react.component
-let make = (~children as _: React.element, ~fallback as _: params<'error> => React.element) =>
-  getErrorBoundary(component)
+@react.component @val
+external make: (
+  ~children: React.element,
+  ~fallback: params<'error> => React.element,
+) => React.element = "ErrorBoundary"