Skip to content

[Feature Request] Improvements to Summary Group Widget #136#518

Draft
michaelbontyes wants to merge 5 commits intounicef-drp:mainfrom
michaelbontyes:main
Draft

[Feature Request] Improvements to Summary Group Widget #136#518
michaelbontyes wants to merge 5 commits intounicef-drp:mainfrom
michaelbontyes:main

Conversation

@michaelbontyes
Copy link

@michaelbontyes michaelbontyes commented Jun 17, 2025

Proposed changes

Feature request details: #136

  • Add more functions (currently, only SUM is available, add: MIN, MAX, AVG, COUNT, COUNT_UNIQUE) - 48a6b00
  • Add option to group values by Admin unit (entity) name (currently, it is only possible to group by geometry_code, as well as concept_uuid, date, value and label) - 722ee50
  • Add Sort By option (currently, the default option is to sort by value. It would be useful to sort data by geographical names rather than codes) and change the sorting mode (asc/desc) - 994446c
  • Add top n filter - to show only top N records (e.g. 5 districts with the highest values for a given indicator) - 63ab255
  • Add option to round values in a smart way (e.g. use 12,345,678 -> 12.34M, 456,123 -> 456.12k, etc) - aa64a1f

Types of changes

What types of changes does your code introduce to Appium?
Put an x in the boxes that apply

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update (if none of the other choices apply)

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

  • I have read the Contributing Guide
  • I have signed the CLA
  • Lint and unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added the necessary documentation (if appropriate)
  • Any dependent changes have been merged and published in downstream modules

Further comments

If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

}
})
return <span>{numberWithCommas(total)} {unit}</span>
case DEFINITION.WidgetOperation.MIN:

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is a separate file for the Summary Group Widget. The feature is confusing, but I think they needed it for the Summary Group Widget

placeholder="Number of top records to show (0 for all)"
/>
</FormControl>
</Fragment>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will get the toggle in UI but you need to add support from where the data is coming from, check this 2dc535f

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

Successfully merging this pull request may close these issues.

2 participants