fix: lint issues

Author: navinkarkera

src/generic/key-utils.test.ts

@@ -169,14 +169,16 @@ describe('component utils', () => {
   });
 
   describe('isContainerType', () => {
-    for (const containerType of [
-      ContainerType.Vertical,
-      ContainerType.Sequential,
-      ContainerType.Chapter,
-      ContainerType.Unit,
-      ContainerType.Subsection,
-      ContainerType.Section,
-    ] as const) {
+    for (
+      const containerType of [
+        ContainerType.Vertical,
+        ContainerType.Sequential,
+        ContainerType.Chapter,
+        ContainerType.Unit,
+        ContainerType.Subsection,
+        ContainerType.Section,
+      ] as const
+    ) {
       it(`returns true for '${containerType}'`, () => {
         expect(isContainerType(containerType)).toBe(true);
       });
@@ -190,30 +192,34 @@ describe('component utils', () => {
   });
 
   describe('isContainerUsageKey', () => {
-    for (const usageKey of [
-      'lct:org:lib:section:my-section-9284e2',
-      'lct:org:lib:subsection:my-subsection-9284e2',
-      'lct:org:lib:unit:my-unit-9284e2',
-      'block-v1:org+type@chapter+block@1',
-      'block-v1:org+type@sequential+block@1',
-      'block-v1:org+type@vertical+block@1',
-      'block-v1:org+type@section+block@1',
-      'block-v1:org+type@subsection+block@1',
-      'block-v1:org+type@unit+block@1',
-    ]) {
+    for (
+      const usageKey of [
+        'lct:org:lib:section:my-section-9284e2',
+        'lct:org:lib:subsection:my-subsection-9284e2',
+        'lct:org:lib:unit:my-unit-9284e2',
+        'block-v1:org+type@chapter+block@1',
+        'block-v1:org+type@sequential+block@1',
+        'block-v1:org+type@vertical+block@1',
+        'block-v1:org+type@section+block@1',
+        'block-v1:org+type@subsection+block@1',
+        'block-v1:org+type@unit+block@1',
+      ]
+    ) {
       it(`returns true for '${usageKey}'`, () => {
         expect(isContainerUsageKey(usageKey)).toBe(true);
       });
     }
 
-    for (const usageKey of [
-      'lb:org:lib:html:id',
-      'block-v1:org+type@problem+block@1',
-      'not a key',
-      '',
-      undefined,
-      null,
-    ]) {
+    for (
+      const usageKey of [
+        'lb:org:lib:html:id',
+        'block-v1:org+type@problem+block@1',
+        'not a key',
+        '',
+        undefined,
+        null,
+      ]
+    ) {
       it(`returns false for '${usageKey}'`, () => {
         expect(isContainerUsageKey(usageKey as any)).toBe(false);
       });

src/library-authoring/LibraryBlock/LibraryBlock.tsx

@@ -53,10 +53,6 @@ export const LibraryBlock = ({
   const intl = useIntl();
   const params = new URLSearchParams();
 
-  if (version === 0) {
-    return null;
-  }
-
   if (version) {
     params.set('version', version.toString());
   }
@@ -86,6 +82,10 @@ export const LibraryBlock = ({
 
   useIframeContent(iframeRef, setIframeRef);
 
+  if (version === 0) {
+    return null;
+  }
+
   return (
     <iframe
       ref={iframeRef}

src/library-authoring/component-comparison/CompareChangesWidget.tsx

@@ -104,17 +104,19 @@ const CompareChangesWidget = ({
 
   return (
     <div className="bg-light-300 py-2 px-1">
-      {sideBySide ? (
-        <Stack direction='horizontal' gap={3}>
-          {oldBlock}
-          {newBlock}
-        </Stack>
-      ) : (
-        <Tabs variant="tabs" defaultActiveKey="new" id="preview-version-toggle" mountOnEnter>
-          {oldBlock && <Tab eventKey="old" title={oldTabMessage}>{oldBlock}</Tab>}
-          <Tab eventKey="new" title={newTabMessage}>{newBlock}</Tab>
-        </Tabs>
-      )}
+      {sideBySide ?
+        (
+          <Stack direction="horizontal" gap={3}>
+            {oldBlock}
+            {newBlock}
+          </Stack>
+        ) :
+        (
+          <Tabs variant="tabs" defaultActiveKey="new" id="preview-version-toggle" mountOnEnter>
+            {oldBlock && <Tab eventKey="old" title={oldTabMessage}>{oldBlock}</Tab>}
+            <Tab eventKey="new" title={newTabMessage}>{newBlock}</Tab>
+          </Tabs>
+        )}
     </div>
   );
 };

src/library-authoring/generic/history-log/HistoryCompareChangesModal.tsx

@@ -43,19 +43,19 @@ const HistoryCompareChangesModal = ({
       isOpen={isOpen}
       onClose={onClose}
       size="xl"
-      className={classNames({'w-xl-100 mw-xl': sideBySide})}
+      className={classNames({ 'w-xl-100 mw-xl': sideBySide })}
       title={title}
       isOverflowVisible={false}
     >
       <ModalDialog.Header>
         <ModalDialog.Title>
-          <Stack direction='horizontal' gap={2}>
+          <Stack direction="horizontal" gap={2}>
             <FormattedMessage
               {...messages.previewChangesTitle}
               values={{
                 title: (
                   <>
-                    <Icon size='lg' src={getItemIcon(blockType)} className="mr-1" />
+                    <Icon size="lg" src={getItemIcon(blockType)} className="mr-1" />
                     {oldTitle}
                   </>
                 ),

src/library-authoring/generic/history-log/HistoryLog.tsx

@@ -151,6 +151,7 @@ export const HistoryContainerLog = ({ containerId }: { containerId: string; }) =
     <div className="history-log">
       {draftHistory && draftHistory.length !== 0 && (
         <HistoryDraftLogGroup
+          itemId={containerId}
           displayName={container?.displayName ?? ''}
           entries={draftHistory}
         />