fix: use Link instead anchor tag

Author: marslanabdulrauf

src/optimizer-page/scan-results/BrokenLinkTable.tsx

@@ -25,6 +25,7 @@ import {
 } from '../../constants';
 import { buildBlockContainerUrl } from '../utils';
 import { useCourseAuthoringContext } from '@src/CourseAuthoringContext';
+import { Link } from 'react-router-dom';
 
 const BrokenLinkHref: FC<{ href: string; }> = ({ href }) => {
   const handleClick = (event: React.MouseEvent<HTMLAnchorElement>) => {
@@ -42,16 +43,12 @@ const BrokenLinkHref: FC<{ href: string; }> = ({ href }) => {
 };
 
 const GoToBlock: FC<{ block: { url: string; displayName?: string; }; }> = ({ block }) => {
-  const handleClick = (event: React.MouseEvent<HTMLAnchorElement>) => {
-    event.preventDefault();
-    window.open(block.url, '_blank');
-  };
 
   return (
     <div className="go-to-block-link-container">
-      <a href={block.url} onClick={handleClick} className="broken-link" rel="noreferrer">
+      <Link to={block.url} className="broken-link" rel="noreferrer" target="_blank">
         {block.displayName}
-      </a>
+      </Link>
     </div>
   );
 };

src/optimizer-page/utils.ts

@@ -1,14 +1,11 @@
-import { createCorrectInternalRoute } from '../utils';
 import { LinkCheckResult } from './types';
 
 export const buildBlockContainerUrl = (
   courseId: string,
   unitId: string,
   blockId: string,
 ): string => {
-  const path = `/course/${courseId}/container/${unitId}#${blockId}`;
-  const internalPath = createCorrectInternalRoute(path);
-  return internalPath;
+  return `/course/${courseId}/container/${unitId}#${blockId}`;
 };
 
 export const countBrokenLinks = (