-
-
Notifications
You must be signed in to change notification settings - Fork 30
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
v1.6.25 - Adds support for distinct sums, averages, and more #587
Conversation
…flagged as distinct - @jongpie FYI
… flag which, when true, de-duplicates children values prior to rolling them up to the parent
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #587 +/- ##
==========================================
- Coverage 96.60% 96.59% -0.01%
==========================================
Files 32 32
Lines 6684 6705 +21
Branches 61 61
==========================================
+ Hits 6457 6477 +20
- Misses 222 223 +1
Partials 5 5
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Exciting! I read the blog post about this refactor 👍 |
@systemsfixer I know it's been a while since we spoke about them, but I'm excited to make progress now! |
Rollup__mdt.IsDistinct__c
. Technically, a CONCAT-based rollup can now supply this flag instead of using theCONCAT_DISTINCT
rollup operation value, but I am leaving support forCONCAT_DISTINCT
in order to not make this a breaking changeRollupCalculator
that was previouslySObject
-based, which will allow me to continue work on a feature that was requested last year (which I have not had time to get back to): the ability to "group" rollup operations (so, taking sums from two different children/two different children fields and applying them to the same parent field, or even wilder: taking the max of two sums and applying the winner to a parent field). While this work is still to come, this release paves the way for that functionality