fix(dashboard): update property name for tables when displaying alarms

awslabs · Apr 7, 2023 · d8d5541 · d8d5541
1 parent 0e90781
commit d8d5541
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/packages/dashboard/src/components/resourceExplorer/components/mapper.spec.ts b/packages/dashboard/src/components/resourceExplorer/components/mapper.spec.ts
@@ -66,7 +66,7 @@ it('can map an asset description to asset summary', () => {
         properties: [
           {
             propertyId: 'e70495ff-c016-4175-9012-62c37857e0d1',
-            name: 'AWS/ALARM_STATE',
+            name: 'windSpeedAlarm',
             unit: undefined,
             alias: undefined,
             dataType: 'STRUCT',

diff --git a/packages/dashboard/src/components/resourceExplorer/components/mapper.ts b/packages/dashboard/src/components/resourceExplorer/components/mapper.ts
@@ -36,7 +36,11 @@ const mapPropertySummary = ({ id, name, unit, dataType, alias }: AssetProperty):
 const mapCompositeModelToAlarmSummary = (model: AssetCompositeModel): AlarmSummary => ({
   name: model.name,
   id: model?.id,
-  properties: model.properties?.filter(isAlarmState).map(mapPropertySummary) ?? [],
+  properties:
+    model.properties
+      ?.filter(isAlarmState)
+      .map(({ id, unit, dataType, alias }) => mapPropertySummary({ id, unit, dataType, alias, name: model.name })) ??
+    [],
 });
 
 export const mapAssetDescriptionToAssetSummary = (description: DescribeAssetResponse): AssetSummary => ({