balderdashy / waterline-docs

WARNING: The content in this repo is out of date! See https://github.com/balderdashy/sails-docs for the most up-to-date documentation
452 stars 161 forks source link

Added docs about .query method #57

Closed Globegitter closed 9 years ago

Globegitter commented 9 years ago

Since I was using it today thought it would be useful. Let me know id anything is missing.

devinivy commented 9 years ago

Thanks! Two things

  1. I think sails-mysql returns the rows directly in results. Maybe we should highlight that the format of results is dependent on the underlying database wrapper. And perhaps we should standardize the results fomat with a breaking change in 0.11 (reflected in docs and SQL interface tests).
  2. There's a tiny markdown typo (missing ]) around the sails-postgresql link.
Globegitter commented 9 years ago

Thanks for pointing out @devinivy Should be both fixed now. And yep I agree standardising that for 0.11 would imo be worth the breaking change.

devinivy commented 9 years ago

:heavy_check_mark: looks good to me! @tjwebb / @dmarcelino ?

tjwebb commented 9 years ago

Thanks @Globegitter !