Merge pull request #161 from zhuje/ou872-modal-pr-2

OU-872: [Part 2] Use modal id when using the modal from the dynamic plugin SDK

Author: openshift-merge-bot[bot]

Makefile

@@ -31,7 +31,7 @@ start-console:
 
 .PHONY: lint-frontend
 lint-frontend:
-	cd web && npm run lint
+	cd web && npm run lint && npm run lint:tsc
 
 .PHONY: install-backend
 install-backend:

web/src/components/Chooser.tsx

@@ -13,12 +13,18 @@ export const Chooser: React.FC<ChooserProps> = ({ selectedID, items, onChange })
     {items.map(({ label, id, tooltip }) => {
       let item = (
         <ToggleGroupItem
+          key={`chooser-togglegroupitem-${id}`}
           text={label}
           isSelected={selectedID === id}
           onChange={(_, on) => on && onChange && onChange(id)}
         />
       );
-      if (tooltip) item = <Tooltip content={tooltip}>{item}</Tooltip>;
+      if (tooltip)
+        item = (
+          <Tooltip key={`chooser-tooltip-${id}`} content={tooltip}>
+            {item}
+          </Tooltip>
+        );
       return item;
     })}
   </ToggleGroup>