From 4e2577191d7abad2daa2402a909d4db09c50c530 Mon Sep 17 00:00:00 2001
From: Bilal Qamar <59555732+BilalQamar95@users.noreply.github.com>
Date: Wed, 24 Aug 2022 17:06:13 +0500
Subject: [PATCH] refactor: resolved eslint issues after merge with master

---
 .../BulkEmailContentHistory.jsx               |  7 +-----
 .../BulkEmailPendingTasksAlert.jsx            | 24 ++++++-------------
 2 files changed, 8 insertions(+), 23 deletions(-)

diff --git a/src/components/bulk-email-tool/bulk-email-task-manager/BulkEmailContentHistory.jsx b/src/components/bulk-email-tool/bulk-email-task-manager/BulkEmailContentHistory.jsx
index 5d6199b..3a32a0a 100644
--- a/src/components/bulk-email-tool/bulk-email-task-manager/BulkEmailContentHistory.jsx
+++ b/src/components/bulk-email-tool/bulk-email-task-manager/BulkEmailContentHistory.jsx
@@ -94,12 +94,6 @@ function BulkEmailContentHistory({ intl }) {
     },
   ];
 
-  const getCell = (tableData, row) => (
-    <Button variant="link" className="px-1" onClick={() => onViewMessageClick(tableData[row.index])}>
-      {intl.formatMessage(messages.buttonViewMessage)}
-    </Button>
-  );
-
   /**
    * Paragon's DataTable supports the ability to add extra columns that might not directly coincide with the data being
    * represented in the table. We are using an additional column to embed a button that will open a Modal to display the
@@ -112,6 +106,7 @@ function BulkEmailContentHistory({ intl }) {
       {
         id: 'view_message',
         Header: '',
+        // eslint-disable-next-line react/no-unstable-nested-components
         Cell: ({ row }) => (
           <Button variant="link" className="px-1" onClick={() => onViewMessageClick(tableData[row.index])}>
             {intl.formatMessage(messages.buttonViewMessage)}
diff --git a/src/components/bulk-email-tool/bulk-email-task-manager/BulkEmailPendingTasksAlert.jsx b/src/components/bulk-email-tool/bulk-email-task-manager/BulkEmailPendingTasksAlert.jsx
index f04268d..881d949 100644
--- a/src/components/bulk-email-tool/bulk-email-task-manager/BulkEmailPendingTasksAlert.jsx
+++ b/src/components/bulk-email-tool/bulk-email-task-manager/BulkEmailPendingTasksAlert.jsx
@@ -15,28 +15,18 @@ export default function BulkEmailPendingTasksAlert() {
       <Hyperlink
         destination={`${getConfig().LMS_BASE_URL}/courses/${window.location.pathname.split('/')[2]}/instructor#view-course-info`}
         target="_blank"
+        isInline
         showLaunchIcon={false}
       >
         <FormattedMessage
           id="bulk.email.pending.tasks.link"
           defaultMessage="Course Info"
         />
-        <Hyperlink
-          destination={`${getConfig().LMS_BASE_URL}/courses/${window.location.pathname.split('/')[2]}/instructor#view-course-info`}
-          target="_blank"
-          isInline
-          showLaunchIcon={false}
-        >
-          <FormattedMessage
-            id="bulk.email.pending.tasks.link"
-            defaultMessage="Course Info"
-          />
-        </Hyperlink>
-        <FormattedMessage
-          id="bulk.email.pending.tasks.description.two"
-          defaultMessage="&nbsp;in the Instructor Dashboard."
-        />
-      </Alert>
-    </>
+      </Hyperlink>
+      <FormattedMessage
+        id="bulk.email.pending.tasks.description.two"
+        defaultMessage="&nbsp;in the Instructor Dashboard."
+      />
+    </Alert>
   );
 }