Jelajahi Sumber

Refactor SelectablePagePageList to use onClickMethodButton prop instead of onClick

Shun Miyazawa 8 bulan lalu
induk
melakukan
d0c635d49d

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

@@ -181,7 +181,7 @@ export const AiAssistantKeywordSearch = (props: { updateBaseSelectedPages: (page
               <SelectablePagePageList
                 pages={pagesWithGlobPath ?? []}
                 method="add"
-                onClick={addPageHandler}
+                onClickMethodButton={addPageHandler}
                 disablePageIds={selectedPages.map(page => page._id)}
               />
             </div>
@@ -198,7 +198,7 @@ export const AiAssistantKeywordSearch = (props: { updateBaseSelectedPages: (page
               <SelectablePagePageList
                 pages={selectedPages}
                 method="remove"
-                onClick={removePageHandler}
+                onClickMethodButton={removePageHandler}
               />
             )
             : (

+ 3 - 3
apps/app/src/features/openai/client/components/AiAssistant/AiAssistantManagementModal/SelectablePagePageList.tsx

@@ -8,12 +8,12 @@ type Props = {
   pages: IPageHasId[],
   method: 'add' | 'remove',
   disablePageIds?: string[],
-  onClick: (page: IPageHasId) => void,
+  onClickMethodButton: (page: IPageHasId) => void,
 }
 
 export const SelectablePagePageList = (props: Props): JSX.Element => {
   const {
-    pages, method, disablePageIds, onClick,
+    pages, method, disablePageIds, onClickMethodButton,
   } = props;
 
   return (
@@ -34,7 +34,7 @@ export const SelectablePagePageList = (props: Props): JSX.Element => {
               disabled={disablePageIds?.includes(page._id)}
               onClick={(e) => {
                 e.stopPropagation();
-                onClick(page);
+                onClickMethodButton(page);
               }}
             >
               <span className="material-symbols-outlined">