e-mission / op-admin-dashboard

An admin/deployer dashboard for the NREL OpenPATH platform
0 stars 9 forks source link

Support including/excluding test users #53

Open shankari opened 1 year ago

shankari commented 1 year ago

Test users are users whose subgroup is test These are typically the admins, who want to check that the app works properly. We want to be able to filter out their data from the visualizations, currently only the map And if we ever to add the ability to visualize the data inline, it would be good to exclude them from those as well

The public dashboard currently does this using a field from the dynamic config The admin dashboard can either use the same field, or provide a checkbox in the selection area to include/exclude test data.

shankari commented 7 months ago

@JGreenlee will work on this

JGreenlee commented 7 months ago

Since our only visualizations currently are for "Map", I was thinking I'd just implement this filter on that page.

But I'm wondering if it would also be valuable to include/exclude test users from all data including the tables on the "Data" page, and the summary on "Overview".

If so, I think a global filter similar to the datepicker would be more appropriate. Thoughts?

shankari commented 7 months ago

I think we should exclude them from all data. Global filter would be best.