Notifications #157

Merged
merged 5 commits into from Jul 6, 2015

Projects

None yet

1 participant

@felixrieseberg
Contributor

Hey @sedouard, I think this looks pretty good - there's a bunch of UI work left to do, tests aren't there yet, bulk & promise notifications aren't implemented yet - but performance right now is awesome.

Plus, I did use Ember.Object, giving us all the benefits of observers and properties 🚀

Closes #150

@felixrieseberg
Contributor

This looks pretty good to me now, I added a few things. I'm bold enough to probably merge this as it is and fix all troubles you have on top of it.

@felixrieseberg felixrieseberg merged commit d729b3b into Azure:master Jul 6, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment