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

Multiple aggregators #218

Closed
bestik opened this issue Sep 28, 2014 · 3 comments
Closed

Multiple aggregators #218

bestik opened this issue Sep 28, 2014 · 3 comments

Comments

@bestik
Copy link

bestik commented Sep 28, 2014

Would it be possible to merge changes mentioned in #30 by javg151 https://github.com/javg151/Projects/tree/master/PivotTable to the current tree? I have tested that and it works well (except that standard version wouldn't need to have a function for comparison of 2 fields). Unfortunately branches are too far away right now. But this would solve a common problem where multiple aggregators are needed.

@nicolaskruchten
Copy link
Owner

Unfortunately the link you mentioned doesn't work for me, but there was a pull request I closed related to this: #111. Is that the code you mean?

@bestik
Copy link
Author

bestik commented Sep 29, 2014

Yes, you are right. That's the code I mean.

@nicolaskruchten
Copy link
Owner

After further thought, I still don't like the approach in PR #111 and the code link above doesn't work, so I'm closing this issue.

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

No branches or pull requests

2 participants