feat: Add descriptive statistics for number cells in data browser

parse-community / parse-dashboard

A dashboard for managing Parse Server

https://parseplatform.org

Other

3.73k stars 1.38k forks source link

feat: Add descriptive statistics for number cells in data browser #2529

Closed AshishBarvaliya closed 4 months ago

AshishBarvaliya commented 4 months ago

New Pull Request Checklist

[x] I am not disclosing a vulnerability.
[x] I am creating this PR in reference to an issue.

Issue Description

Closes: #2506

Approach

Holding down the Shift key while clicking will conveniently highlight the entire area.
Currently, I'm displaying statistics in the toolbar using a default theme. Feel free to share your preferences.
By clicking on the stats button, users can opt for different types of statistics.
If any selected column isn't categorized as "Number," it won't be displayed or included in calculations.
Null or undefined values will be disregarded, ensuring they don't impact the count or any statistics.

ss1:

ss2:

parse-github-assistant[bot] commented 4 months ago

Thanks for opening this pull request!

🎉 We are excited about your hands-on contribution!

uffizzi-cloud[bot] commented 4 months ago

Uffizzi Ephemeral Environment `deployment-47177`

:watch: Updated Feb 23, 2024, 18:38 UTC

:cloud: https://app.uffizzi.com/github.com/parse-community/parse-dashboard/pull/2529

:page_facing_up: View Application Logs etc.

What is Uffizzi? Learn more

mtrezza commented 4 months ago

Wow! This is a truly impressive PR!

Could the selected cells have a highlighted cell background color, so that it's easier to identify the selected cells visually? Similar to how you'd see it in a spreadsheet UI.

AshishBarvaliya commented 4 months ago

I've added a lighter shade of blue to ensure that the text and background have sufficient contrast.

Let me know if any alternative styles are needed for the stats menu or if you'd like to add any other types of stats.

uffizzi-cloud[bot] commented 4 months ago

Uffizzi Ephemeral Environment `deployment-47206`

:watch: Updated Feb 26, 2024, 00:44 UTC

:cloud: https://app.uffizzi.com/github.com/parse-community/parse-dashboard/pull/2529

:page_facing_up: View Application Logs etc.

What is Uffizzi? Learn more

AshishBarvaliya commented 4 months ago

I found that if there is a string column within the selection, the stats field won't be displayed at all. Is that intended, or should it just be ignored in the calculation? For example, this selection doesn't display anything:

Yes, this intentional behaviour is designed to prevent confusion. Currently, only 'Number' types are allowed. We can consider adding exceptions for strings if necessary or requested, but for now, it's not required.

I've also addressed the issue with the jobs that were failing.

uffizzi-cloud[bot] commented 4 months ago