Breeze / breeze.js.samples

Breeze JavaScript Client sample applications
MIT License
96 stars 85 forks source link

Remove unnecessary use of TaskCompletionSource<T>. #24

Closed lothrop closed 9 years ago

lothrop commented 9 years ago

The use of TaskCompletionSource in the examples in unnecessarily complicated. Task.FromResult() is a better fit.

wardbell commented 9 years ago

Thanks. This would be an improvement ... except that I've decided to switch to Web API's own approved CORS package rather than roll my own (which I had to do until Web API offered one).

This is the reason I'm rejecting your otherwise valuable pull request.

Appreciate it though.