Skip to content

Merge vacance metrics data in metrics list

Sylvain Boulade requested to merge acv-vacance-metrics-display into develop

Context

For ACV vacance metrics, we have two distinct metrics:

  • One returning the percentage per city for a given territory
  • One returning the average percentage for the territory

However per ACV maintainer's request, we only want to display one metric, showing the average as "value" and having a button to display the detailed list (the current "value" for the detailed list is redundant / not useful).

At the moment, insitu cannot merge these two metrics into one as for "rows" type metrics, it can only provide the number of elements in the list as "count". and cannot give us other arbitrary values (like the average we want here).

For this reason and until there is an alternative backend side, we are "overriding" the data for the detailed metric with the average instead of the count manually.

Content

This is a very dirty workaround, but afaik there is no better way until we have a more robust solution backend side.

Merge request reports