fix: lint issues

Author: navinkarkera

src/course-outline/data/api.ts

@@ -267,13 +267,20 @@ export async function configureCourseSubsection(
   } = variables;
 
   const metadata = pickDefined({
-    visible_to_staff_only: isVisibleToStaffOnly === undefined ? undefined : isVisibleToStaffOnly ? true : null,
+    visible_to_staff_only: (() => {
+      if (isVisibleToStaffOnly === undefined) {
+        return undefined;
+      }
+      return isVisibleToStaffOnly ? true : null;
+    })(),
     due: dueDate,
     hide_after_due: hideAfterDue,
     show_correctness: showCorrectness,
     is_practice_exam: isPracticeExam,
     is_time_limited: isTimeLimited,
-    is_proctored_enabled: (isProctoredExam !== undefined || isPracticeExam !== undefined || isOnboardingExam !== undefined)
+    is_proctored_enabled: (
+      isProctoredExam !== undefined || isPracticeExam !== undefined || isOnboardingExam !== undefined
+    )
       ? (isProctoredExam || isPracticeExam || isOnboardingExam)
       : undefined,
     exam_review_rules: examReviewRules,

src/course-outline/outline-sidebar/info-sidebar/SubsectionSettings.test.tsx

@@ -130,7 +130,9 @@ describe('SubsectionSettings', () => {
   it('handles grading select and due date/time changes', async () => {
     apiHooks.useCourseDetails.mockReturnValue({ data: { selfPaced: false } });
     apiHooks.useCourseItemData.mockReturnValue({
-      data: { ...baseItemData, graded: false, prereqMinScore: '50', prereqMinCompletion: '75' },
+      data: {
+        ...baseItemData, graded: false, prereqMinScore: '50', prereqMinCompletion: '75',
+      },
       isPending: false,
     });
 
@@ -178,7 +180,14 @@ describe('SubsectionSettings', () => {
 
   it('does not call mutate when item data is pending', async () => {
     apiHooks.useCourseDetails.mockReturnValue({ data: { selfPaced: false } });
-    apiHooks.useCourseItemData.mockReturnValue({ data: { ...baseItemData, start: null, graded: false }, isPending: true });
+    apiHooks.useCourseItemData.mockReturnValue({
+      data: {
+        ...baseItemData,
+        start: null,
+        graded: false,
+      },
+      isPending: true,
+    });
 
     const user = userEvent.setup();
     render(<SubsectionSettings subsectionId={subsectionId} />);
@@ -189,7 +198,9 @@ describe('SubsectionSettings', () => {
 
   it('resets grading local state when itemData changes', async () => {
     apiHooks.useCourseDetails.mockReturnValue({ data: { selfPaced: false } });
-    const firstItemData = { ...baseItemData, format: 'g1', due: '2024-01-01', graded: true };
+    const firstItemData = {
+      ...baseItemData, format: 'g1', due: '2024-01-01', graded: true,
+    };
     const secondItemData = { ...firstItemData, format: 'g2', due: '2024-02-02' };
 
     apiHooks.useCourseItemData.mockReturnValue({ data: firstItemData, isPending: false });

src/course-outline/outline-sidebar/info-sidebar/SubsectionSettings.tsx

@@ -11,7 +11,9 @@ import AdvancedTab from '@src/generic/configure-modal/AdvancedTab';
 import { DatepickerControl, DATEPICKER_TYPES } from '@src/generic/datepicker-control';
 import { SidebarContent, SidebarSection } from '@src/generic/sidebar';
 import { useStateWithCallback } from '@src/hooks';
-import { useCallback, useEffect, useRef, useState } from 'react';
+import {
+  useCallback, useEffect, useRef, useState,
+} from 'react';
 import { useSelector } from 'react-redux';
 import { ReleaseSection } from './sharedSettings/ReleaseSection';
 import messages from './messages';