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

Add Group By Pipe #8

Merged
merged 1 commit into from
Dec 9, 2016
Merged

Add Group By Pipe #8

merged 1 commit into from
Dec 9, 2016

Conversation

paulussup
Copy link
Contributor

Didn't see anything like this. I find it is useful.

@coveralls
Copy link

coveralls commented Dec 9, 2016

Coverage Status

Coverage remained the same at 100.0% when pulling 7a31659 on NextFaze:master into 2a2c9bb on danrevah:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 7a31659 on NextFaze:master into 2a2c9bb on danrevah:master.

@danrevah danrevah merged commit 5ba7c15 into danrevah:master Dec 9, 2016
@danrevah
Copy link
Owner

danrevah commented Dec 9, 2016

Looks good .. Thanks!
I will add a documentation for it.

@danrevah
Copy link
Owner

danrevah commented Dec 9, 2016

Added to version #v0.5.2 with small changes in behaviour, it will now return an object that it's property will be the grouped by property.

Take a look at the readme file for more documentation, if you would like to use it in an ngFor you could add an additional keys pipe to loop over it, since Angular2 doesn't support loop over objects.

@paulussup
Copy link
Contributor Author

paulussup commented Dec 9, 2016 via email

@danrevah
Copy link
Owner

Might be a good idea yeah.
I'll try to find a time for it.

@danrevah
Copy link
Owner

@paulussup Added some object pipes: https://github.com/danrevah/ng2-pipes#object

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

Successfully merging this pull request may close these issues.

3 participants