davidchang/ngReact — pull request #11 by thenikso (2014-10-22 09:21:49+00:00)
Title: Re-implement reactDirective to be more angular-oriented

With reference to issue #10, this PR has the following goals:

- Re-implement `reactDirective` to accept a React component instead of an injectable name as it's first parameter
- Re-implement `reactComponent` to use `reactDirective`

This PR contains **breaking changes** to the current usage of `reactDirective`. It also requires AngularJS 1.3+

**TODO:**

[x] `reactDirective` should accept a React component as it's first parameter
[ ] `reactDirective` should sanity check it's parameters
[x] `reactDirective` should update the view when changes in the angular models occurs
[ ] `reactComponent` should be implemented using `reactDirective`
[ ] React components rendered via `reactDirective` should have the means to update angular models when updating React props