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

Is there any plan on supporting multiple fields for the 'groupBy' filter? #19

Closed
shikhartanwar opened this issue Jan 26, 2017 · 2 comments

Comments

@shikhartanwar
Copy link

This is not an issue but more of a feature request. The groupBy filter in angular1 supported grouping on multiple fields and generally also it is used for grouping on multiple fields only. Are there any plans in the near future to support that?

@shikhartanwar shikhartanwar changed the title Is there any plan on supporting multiple fields on the groupBy filter? Is there any plan on supporting multiple fields for the 'groupBy' filter? Jan 26, 2017
@danrevah
Copy link
Owner

danrevah commented Jan 26, 2017

I haven't saw such a behaviour in the angular1 filter, but since it's a good idea and it was quite easy to add. I've just released it in version v1.3.0.

It now supports multiple fields and nested properties.
Take a look at the documentation: https://github.com/danrevah/ngx-pipes#groupby

Let me know if you have any other ideas on improving the functionality of the pipes.
I'm always open for additional functionality which will give answers to more use-cases.

Thanks.

@shikhartanwar
Copy link
Author

Awesome!! Thanks for the prompt reply and release. This is really helpful. The groupBy pipe works like a charm for multiple fields as well now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants