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

aggregation support for doctrine source #513

Merged
merged 1 commit into from Apr 10, 2017

Conversation

2 participants
@chapcz
Copy link
Contributor

chapcz commented Apr 9, 2017

No description provided.

@juniwalk juniwalk merged commit 922a6f1 into contributte:v5.x Apr 10, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@juniwalk

This comment has been minimized.

Copy link
Collaborator

juniwalk commented Apr 10, 2017

Thank you

@juniwalk

This comment has been minimized.

Copy link
Collaborator

juniwalk commented Apr 10, 2017

You could have add querybuilder to default SUM function when I think about it. Will you finish it? :)

See https://github.com/ublaboo/datagrid/blob/v5.x/src/AggregationFunction/FunctionSum.php#L47

@chapcz

This comment has been minimized.

Copy link
Contributor Author

chapcz commented Apr 10, 2017

Like this? #514

@chapcz chapcz deleted the chapcz:patch-1 branch Sep 9, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session.