Browse Source

resolve warning

Shun Miyazawa 1 year ago
parent
commit
9d0e45d6ea

+ 1 - 1
apps/app/src/features/openai/client/components/AiAssistant/AiAssistantManagementModal/AccessScopeDropdown.tsx

@@ -47,7 +47,7 @@ export const AccessScopeDropdown: React.FC<Props> = (props: Props) => {
         </DropdownToggle>
         </DropdownToggle>
         <DropdownMenu>
         <DropdownMenu>
           { [AiAssistantAccessScope.OWNER, AiAssistantAccessScope.GROUPS, AiAssistantAccessScope.PUBLIC_ONLY].map(accessScope => (
           { [AiAssistantAccessScope.OWNER, AiAssistantAccessScope.GROUPS, AiAssistantAccessScope.PUBLIC_ONLY].map(accessScope => (
-            <DropdownItem onClick={() => selectAccessScopeHandler(accessScope)}>
+            <DropdownItem onClick={() => selectAccessScopeHandler(accessScope)} key={accessScope}>
               {getAccessScopeLabel(accessScope)}
               {getAccessScopeLabel(accessScope)}
             </DropdownItem>
             </DropdownItem>
           ))}
           ))}

+ 2 - 2
apps/app/src/features/openai/client/components/AiAssistant/AiAssistantManagementModal/SelectUserGroupModal.tsx

@@ -37,10 +37,10 @@ const SelectUserGroupModalSubstance: React.FC<Props> = (props: Props) => {
         <button
         <button
           className="btn btn-outline-primary d-flex justify-content-start mb-3 mx-4 align-items-center p-3"
           className="btn btn-outline-primary d-flex justify-content-start mb-3 mx-4 align-items-center p-3"
           type="button"
           type="button"
-          key={userGroup.item.id}
+          key={userGroup.item._id}
           onClick={() => onSelect(userGroup, selectedUserGroupType)}
           onClick={() => onSelect(userGroup, selectedUserGroupType)}
         >
         >
-          <input type="checkbox" checked={checked(userGroup)} />
+          <input type="checkbox" checked={checked(userGroup)} onChange={() => {}} />
           <p className="ms-3 mb-0">{userGroup.item.name}</p>
           <p className="ms-3 mb-0">{userGroup.item.name}</p>
           {userGroup.type === GroupType.externalUserGroup && <span className="ms-2 badge badge-pill badge-info">{userGroup.item.provider}</span>}
           {userGroup.type === GroupType.externalUserGroup && <span className="ms-2 badge badge-pill badge-info">{userGroup.item.provider}</span>}
           {/* TODO: Replace <div className="small">(TBD) List group members</div> */}
           {/* TODO: Replace <div className="small">(TBD) List group members</div> */}