fix: lint issues

Author: navinkarkera

src/container-comparison/CompareContainersWidget.tsx

@@ -13,6 +13,7 @@ import { LoadingSpinner } from '@src/generic/Loading';
 import { useContainer, useContainerChildren } from '@src/library-authoring/data/apiHooks';
 import { BoldText } from '@src/utils';
 
+import { Container, LibraryBlockMetadata } from '@src/library-authoring/data/api';
 import ChildrenPreview from './ChildrenPreview';
 import ContainerRow from './ContainerRow';
 import { useCourseContainerChildren } from './data/apiHooks';
@@ -21,7 +22,6 @@ import {
 } from './types';
 import { diffPreviewContainerChildren, isRowClickable } from './utils';
 import messages from './messages';
-import { Container, LibraryBlockMetadata } from '@src/library-authoring/data/api';
 
 interface ContainerInfoProps {
   upstreamBlockId: string;

src/library-authoring/common/context/SidebarContext.tsx

@@ -204,7 +204,7 @@ export const SidebarProvider = ({
     if (selectedItemId) {
       // if a item is selected that means we have list of items displayed
       // which means we can get the index from url and set it.
-      const indexNumber = indexParam ? Number(indexParam): undefined;
+      const indexNumber = indexParam ? Number(indexParam) : undefined;
       if (selectedItemId.startsWith('lct:')) {
         openContainerInfoSidebar(selectedItemId, indexNumber);
       } else if (selectedItemId.startsWith('lb:')) {

src/library-authoring/components/ComponentRemover.tsx

@@ -86,7 +86,7 @@ const ComponentRemover = ({ usageKey, index, close }: Props) => {
       });
     };
     updateContainerChildrenMutation.mutateAsync(updatedKeys).then(() => {
-      if (sidebarItemInfo?.id === usageKey && sidebarItemInfo?.index  === index) {
+      if (sidebarItemInfo?.id === usageKey && sidebarItemInfo?.index === index) {
         // Close sidebar if current component is open
         closeLibrarySidebar();
       }
@@ -102,7 +102,7 @@ const ComponentRemover = ({ usageKey, index, close }: Props) => {
     });
 
     close();
-  }
+  };
 
   const removeText = intl.formatMessage(messages.removeComponentConfirm, {
     componentName: <b>{component?.displayName}</b>,
@@ -117,7 +117,7 @@ const ComponentRemover = ({ usageKey, index, close }: Props) => {
       title={intl.formatMessage(messages.removeComponentWarningTitle)}
       icon={Warning}
       description={removeText}
-      onDeleteSubmit={hasDuplicates ? excludeOneInstance: removeFromContainer}
+      onDeleteSubmit={hasDuplicates ? excludeOneInstance : removeFromContainer}
       btnLabel={intl.formatMessage(messages.componentRemoveButtonLabel)}
       buttonVariant="primary"
     />

src/library-authoring/containers/ContainerRemover.tsx

@@ -9,7 +9,9 @@ import { getBlockType } from '@src/generic/key-utils';
 
 import { useSidebarContext } from '../common/context/SidebarContext';
 import { useLibraryContext } from '../common/context/LibraryContext';
-import { useContainer, useContainerChildren, useRemoveContainerChildren, useUpdateContainerChildren } from '../data/apiHooks';
+import {
+  useContainer, useContainerChildren, useRemoveContainerChildren, useUpdateContainerChildren,
+} from '../data/apiHooks';
 import messages from '../components/messages';
 import { Container } from '../data/api';
 

src/library-authoring/data/api.mocks.ts

@@ -635,7 +635,7 @@ export async function mockGetContainerChildren(containerId: string): Promise<api
     name = blockType;
     typeNamespace = 'lct';
   }
-  let result =Array(numChildren).fill(mockGetContainerChildren.childTemplate).map((child, idx) => (
+  let result = Array(numChildren).fill(mockGetContainerChildren.childTemplate).map((child, idx) => (
     {
       ...child,
       // Generate a unique ID for each child block to avoid "duplicate key" errors in tests

src/library-authoring/data/apiHooks.ts

@@ -736,29 +736,32 @@ export const useRestoreContainer = (containerId: string) => {
 /**
  * Get the metadata and children for a container in a library
  */
-export const useContainerChildren = <T extends { id: string, isNew?: boolean }>(containerId?: string, published: boolean = false) => (
-  useQuery({
-    enabled: !!containerId,
-    queryKey: libraryAuthoringQueryKeys.containerChildren(containerId!),
-    queryFn: () => api.getLibraryContainerChildren<T>(containerId!, published),
-    structuralSharing: (oldData: T[], newData: T[]) => {
+export const useContainerChildren = <T extends { id: string, isNew?: boolean }>(
+  containerId?: string,
+  published: boolean = false,
+) => (
+    useQuery({
+      enabled: !!containerId,
+      queryKey: libraryAuthoringQueryKeys.containerChildren(containerId!),
+      queryFn: () => api.getLibraryContainerChildren<T>(containerId!, published),
+      structuralSharing: (oldData: T[], newData: T[]) => {
       // This just sets `isNew` flag to new children components
-      if (oldData) {
-        const oldDataIds = oldData.map((obj) => obj.id);
-        // eslint-disable-next-line no-param-reassign
-        newData = newData.map((newObj) => {
-          if (!oldDataIds.includes(newObj.id)) {
+        if (oldData) {
+          const oldDataIds = oldData.map((obj) => obj.id);
+          // eslint-disable-next-line no-param-reassign
+          newData = newData.map((newObj) => {
+            if (!oldDataIds.includes(newObj.id)) {
             // Set isNew = true if we have new child on refetch
             // eslint-disable-next-line no-param-reassign
-            newObj.isNew = true;
-          }
-          return newObj;
-        });
-      }
-      return replaceEqualDeep(oldData, newData);
-    },
-  })
-);
+              newObj.isNew = true;
+            }
+            return newObj;
+          });
+        }
+        return replaceEqualDeep(oldData, newData);
+      },
+    })
+  );
 
 /**
  * If you work with `useContentFromSearchIndex`, you can use this

src/library-authoring/section-subsections/LibraryContainerChildren.tsx

@@ -42,7 +42,9 @@ interface ContainerRowProps extends LibraryContainerChildrenProps {
   index?: number;
 }
 
-const ContainerRow = ({ containerKey, container, readOnly, index }: ContainerRowProps) => {
+const ContainerRow = ({
+  containerKey, container, readOnly, index,
+}: ContainerRowProps) => {
   const intl = useIntl();
   const { showToast } = useContext(ToastContext);
   const updateMutation = useUpdateContainer(container.originalId, containerKey);
@@ -166,7 +168,11 @@ export const LibraryContainerChildren = ({ containerKey, readOnly }: LibraryCont
     return setOrderedChildren(newChildren || []);
   }, [children, setOrderedChildren]);
 
-  const handleChildClick = useCallback((child: LibraryContainerMetadataWithUniqueId, numberOfClicks: number, index: number) => {
+  const handleChildClick = useCallback((
+    child: LibraryContainerMetadataWithUniqueId,
+    numberOfClicks: number,
+    index: number,
+  ) => {
     if (readOnly) {
       // don't allow interaction if rendered as preview
       return;

src/library-authoring/units/LibraryUnitBlocks.tsx

@@ -126,7 +126,9 @@ const BlockHeader = ({ block, index, readOnly }: ComponentBlockProps) => {
 };
 
 /** ComponentBlock to render preview of given component under Unit */
-const ComponentBlock = ({ block, readOnly, isDragging, index }: ComponentBlockProps) => {
+const ComponentBlock = ({
+  block, readOnly, isDragging, index,
+}: ComponentBlockProps) => {
   const { showOnlyPublished, openComponentEditor } = useLibraryContext();
 
   const { sidebarItemInfo, openItemSidebar } = useSidebarContext();