mobxjs / mobx-angular

The MobX connector for Angular.
MIT License
483 stars 59 forks source link

be more relaxed about angular version #2

Closed bb closed 7 years ago

bb commented 7 years ago

ng2-mobx seems to run fine with angular 2.1

adamkleingit commented 7 years ago

Thanks for the PRs :) Are you using the library? I would love some feedback

bb commented 7 years ago

Thanks for merging. No, I'm not actively using it because I don't currently use Angular. I just pitched Mobx to a friend who then asked for Angular bindings. So we decided to give ng2-mobx a try.

It worked great out of the box, except for the two tiny things which you merged. So, all feedback is already incorporated.

adamkleingit commented 7 years ago

Great. If there's anything you think I should add to the library down the road I'd love to hear.

All the best

On Sun, Nov 6, 2016 at 1:11 PM, Benjamin Bock notifications@github.com wrote:

Thanks for merging. No, I'm not actively using it because I don't currently use Angular. I just pitched Mobx to a friend who then asked for Angular bindings. So we decided to give ng2-mobx a try.

It worked great out of the box, except for the two tiny things which you merged. So, all feedback is already incorporated.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/500tech/ng2-mobx/pull/2#issuecomment-258674113, or mute the thread https://github.com/notifications/unsubscribe-auth/AA2SSl7I7b1Ma55CQpWUqkNR3aWF_mqQks5q7bXTgaJpZM4Kn3k7 .