@validate already makes a coroutine; 1.2.0 release changes likely unnecessary

hfaran / Tornado-JSON

A simple JSON API framework based on Tornado

http://tornado-json.readthedocs.org/

MIT License

273 stars 60 forks source link

@validate already makes a coroutine; 1.2.0 release changes likely unnecessary #65

Closed hfaran closed 8 years ago

hfaran commented 9 years ago

[ ] Should be added in documentation that if a method is decorated using schema.validate than it can be used asynchronously already.
[ ] Addition of tornado_json.gen.coroutine should be reverted (as should all changes to docs etc. but tests should be kept for testing the breaks in #59)

alexzrh commented 9 years ago

Hi, I tried this on 1.2.0, but schema.validate did not allow the method to be used asynchronously. Once I added tornado_json.gen.coroutine to the method, it worked asynch.