CCM-10917 Tidy up comments

Author: nicki-nhs

frontend/src/__tests__/utils/markdownit/__snapshots__/index.test.tsx.snap

@@ -72,16 +72,4 @@ End of horizontal line</p>
 <p>This is a paragraph</p>
 <p>By doing a new line this starts a new paragraph</p>
 "
-`;
-
-exports[`withEmbeddedLink should only process links and line breaks rules 1`] = `
-"
-# Heading<br>## sub heading<br>**bold**<br>* bullet point 1
-* bullet point 2<br>1. ordered list item 1
-2. ordered list item 2<br><a href="https://www.nhs.uk/example" target="_blank" rel="noopener noreferrer">Read more</a><br>https://www.nhs.uk/example<br>Before page break
-***
-After page break<br>Start of line break<br>end of line break<br>Before horizontal line
----
-End of horizontal line<br>This is a paragraph<br>By doing a new line this starts a new paragraph
-"
-`;
+`;

frontend/src/__tests__/utils/markdownit/index.test.tsx

@@ -3,7 +3,6 @@ import {
   renderEmailMarkdown,
   renderNHSAppMarkdown,
   renderSMSMarkdown,
-  withEmbeddedLink,
 } from '@utils/markdownit';
 import { mockDeep } from 'jest-mock-extended';
 import { markdown } from '../../components/forms/fixtures';
@@ -80,24 +79,3 @@ describe('renderSMSMarkdown', () => {
     expect(renderSMSMarkdown(markdown)).toMatchSnapshot();
   });
 });
-
-describe('withEmbeddedLink', () => {
-  beforeEach(jest.resetAllMocks);
-
-  it('should enable links', () => {
-    const markdownItWrapperMock = mockDeep<MarkdownItWrapper>();
-
-    markdownItWrapperMock.enableLineBreak.mockReturnValue(
-      markdownItWrapperMock
-    );
-
-    withEmbeddedLink('message with a [link](link.com)', markdownItWrapperMock);
-
-    expect(markdownItWrapperMock.enableLineBreak).toHaveBeenCalled();
-    expect(markdownItWrapperMock.enable).toHaveBeenCalledWith('link');
-  });
-
-  it('should only process links and line breaks rules', () => {
-    expect(withEmbeddedLink(markdown)).toMatchSnapshot();
-  });
-});

frontend/src/components/molecules/MarkdownContent/MarkdownContent.tsx

@@ -1,4 +1,3 @@
-// import ReactMarkdown from 'react-markdown';
 import Markdown from 'markdown-to-jsx';
 import React from 'react';
 
@@ -9,19 +8,6 @@ export function MarkdownContent({ segments }: MarkdownContentProps) {
     <>
       {segments.map((content, index) => {
         return (
-          // <ReactMarkdown
-          //   key={index}
-          //   children={content}
-          //   allowedElements={['a', 'p']}
-          //   components={{
-          //     a(props) {
-          //       const { node, ...rest } = props;
-          //       return (
-          //         <a {...rest} target='_blank' rel='noopener noreferrer' />
-          //       );
-          //     },
-          //   }}
-          // />
           <Markdown
             key={index}
             options={{

frontend/src/components/molecules/TemplateSubmitted/TemplateSubmitted.tsx

@@ -3,7 +3,6 @@
 import Link from 'next/link';
 import { NHSNotifyMain } from '@atoms/NHSNotifyMain/NHSNotifyMain';
 import content from '@content/content';
-// import { withEmbeddedLink } from '@utils/markdownit';
 import React from 'react';
 import { MarkdownContent } from '@molecules/MarkdownContent/MarkdownContent';
 

frontend/src/content/content.ts

@@ -686,7 +686,6 @@ const templateSubmitted = {
       ],
     },
   ],
-  // doNext: "### If you've sent messages using NHS Notify\n\n[Raise a request with the Service Desk (opens in a new tab)](https://nhsdigitallive.service-now.com/csm?id=sc_cat_item&sys_id=ce81c3ae1b1c5190892d4046b04bcb83) once you've submitted all your templates.\n\nIf you replaced a template by submitting a new one, tell us which template you want to use in your Service Desk request.",
   backLinkText: backToAllTemplates,
 };
 

frontend/src/utils/markdownit/index.ts

@@ -50,11 +50,3 @@ export function renderEmailMarkdown(
 
   return markdown.render(value);
 }
-
-export function withEmbeddedLink(
-  content: string,
-  markdown = new MarkdownItWrapper()
-) {
-  markdown.enableLineBreak().enable('link');
-  return markdown.renderInline(content);
-}