Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Aggregators are not documented #6309

Closed
stevesims opened this issue Dec 18, 2013 · 3 comments
Closed

Aggregators are not documented #6309

stevesims opened this issue Dec 18, 2013 · 3 comments

Comments

@stevesims
Copy link

Whilst it appears that basic aggregators may be supported by Waterline, as referenced in issue 61, this functionality does not appear to be documented anywhere.

@particlebanana
Copy link
Contributor

Yes these are there and are not currently documented at all as far as I know. @uncletammy has been working on getting all the Sails docs updated and we can get aggregations documented in there.

For now you can look through the tests if you need to implement them. They should work in most/all adapters.

Query Modifier Tests

@mikermcneil
Copy link
Member

@stevesims Excellent point, thanks for bringing it up :) We're planning on pulling in polyfill support for these methods into waterline-criteria for the adapters which don't support them (e.g. sails-rest, sails-redis, sails-disk, sails-memory). waterline-criteria will eventually be merged into waterline core (fingers crossed) and will allow us to provide polyfill support for the other features necessary to implement the queryable interface.

If you have time to put together a few example use cases when you'd want to use aggregators and/or groupBy, it'd help us a lot since we could cannibalize those examples and it would be more like answering questions than writing a little minibook, which starting from scratch on docs can feel like sometimes :\

Thanks again!

@particlebanana
Copy link
Contributor

Moving this to waterline-docs

@johnabrams7 johnabrams7 transferred this issue from balderdashy/waterline May 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants