Fix metrics (#558)

- metrics had bad getId/idAttribute
- better to add custom getId which matches the same process to create the guid in MetricsAction
  however the entity doesn't contain all the info required
- solution is to be more specific with workaround

src/frontend/packages/store/src/normalizr/normalizr.js

@@ -561,7 +561,13 @@ var getUnvisit = function getUnvisit(entities) {
 
     if (schema instanceof EntitySchema) {
       if (typeof input === 'object' && schema.getId) {
-        input = schema.getId(input);
+        const id = schema.getId(input); // Some parts of the console don't specify the correct `getId` or `idAttribute` (latter is used by
+        // default `idAttribute`) i.e. `MetricSchema`. This causes a miss-match between the guid in the action (i.e. MetricsAction) and what
+        // we cache it by. It's better to fix this by a custom `getId`, but for metrics case there's not enough info within the entity to
+        // get it's id
+        if (id) {
+          input = id;
+        }
       }
 
       return unvisitEntity(input, schema, unvisit, getEntity, cache);