Endringer for forberedelse til PR

navikt · lotorvik · Feb 17, 2025 · Feb 17, 2025 · Feb 17, 2025 · Feb 17, 2025
commit 5ed31b3fc307081989750104527d2163ae0ee04d
@@ -30,10 +30,8 @@ const parseBody = (
 };
 
 export const getPdf = async (req: Request) => {
-  logger.info('GetPdf: start');
   const { form, submission, submissionMethod, translations, language } = parseBody(req);
 
-  logger.info(`GetPdf: with submission set = ${submission != undefined}`);
   if (!['nb-NO', 'nn-NO', 'en'].includes(language)) {
     logger.warn(`Language code "${language}" is not supported. Language code will be defaulted to "nb".`);
   }

diff --git a/packages/fyllut-backend/src/routers/api/forstesideV2.test.ts b/packages/fyllut-backend/src/routers/api/forstesideV2.test.ts
@@ -32,6 +32,7 @@ const addresses = [
     },
   },
 ];
+
 const formTitle = 'testskjema';
 const filePathForsteside = path.join(process.cwd(), '/src/routers/api/test-forsteside.pdf');
 const filePathSoknad = path.join(process.cwd(), '/src/routers/api/test-skjema.pdf');
@@ -84,7 +85,6 @@ describe('[endpoint] forsteside', () => {
           components: [],
           properties: { mottaksadresseId: 'mottaksadresseId', path: '12345', skjemanummer: 'NAV 12.34-56' },
         }),
-        submissionData: '{}',
         submissionMethod: 'paper',
         language: 'nb-NO',
         submission: JSON.stringify({ data: {} }),

@@ -22,13 +22,10 @@ const { skjemabyggingProxyUrl, formsApiUrl } = config;
 const forstesideV2 = {
   post: async (req: Request, res: Response, next: NextFunction) => {
     try {
-      logger.info('ForstesideV2');
-      const { form, submissionData, language, enhetNummer } = req.body;
+      const { form, submission, language, enhetNummer } = req.body;
       const formParsed = JSON.parse(form);
-      const submissionDataParsed = JSON.parse(submissionData);
-
-      logger.info('Skal kalle getRecipients');
-
+      const submissionParsed = JSON.parse(submission);
+      const submissionDataParsed = submissionParsed.data;
       const recipients = await getRecipients(formParsed?.properties);
 
       const forstesideBody = forstesideUtils.genererFoerstesideData(
@@ -44,7 +41,6 @@ const forstesideV2 = {
 
       const forstesidePdf = base64Decode(forstesideResponse.foersteside);
 
-      logger.info('Skal kall getPdf');
       const soknadResponse: any = await getPdf(req);
 
       const soknadPdf = base64Decode(soknadResponse.data);

diff --git a/packages/fyllut-backend/src/routers/api/helpers/gotenbergService.ts b/packages/fyllut-backend/src/routers/api/helpers/gotenbergService.ts
@@ -104,6 +104,7 @@ export const buildHtmlAndConvertToPdf = async (
   }
 };
 
+// Sette opp formdata med html og footer for å generere PDF v.hj.a Gotenberg.
 const createPdfFromHtml = async (
   html: string,
   footer: string,
@@ -132,6 +133,7 @@ const createPdfFromHtml = async (
   return await callGotenberg('/forms/chromium/convert/html', formData);
 };
 
+// Sette opp formdata til å merge en liste av PDFer
 export const mergeFiles = async (
   schema: string,
   title: string,
@@ -192,6 +194,7 @@ const applyMetadata = async (
 };
 */
 
+// Generisk metode for kall til mot Gotenberg gitt rute og preparert FormData
 const callGotenberg = async (route: string, formData: FormData): Promise<any> => {
   console.log(`Calling Gotenberg with url = ${gotenbergUrl}${route}`);
 

@@ -42,7 +42,6 @@ const LetterDownload = ({ form, index, submission, enhetsListe, fyllutBaseURL, t
         id={`forsteside-${form.path}`}
         values={{
           form: JSON.stringify(form),
-          submissionData: JSON.stringify(submission.data),
 
           submission: JSON.stringify(submission),
           translations: JSON.stringify(currentLanguage !== 'nb-NO' ? translations[currentLanguage] : {}),
@@ -60,17 +59,6 @@ const LetterDownload = ({ form, index, submission, enhetsListe, fyllutBaseURL, t
           }
         }}
       />
-      <DownloadPdfButton
-        id={`soknad-${form.path}`}
-        values={{
-          form: JSON.stringify(form),
-          submission: JSON.stringify(submission),
-          translations: JSON.stringify(currentLanguage !== 'nb-NO' ? translations[currentLanguage] : {}),
-          language: currentLanguage,
-        }}
-        actionUrl={`${fyllutBaseURL}/api/pdf/convert`}
-        label={translate(form.properties.downloadPdfButtonText || TEXTS.grensesnitt.downloadApplication)}
-      />
     </section>
   );
 };

@@ -152,11 +152,11 @@ describe('PrepareLetterPage', () => {
     it('Laster ned pdf for førsteside', async () => {
       renderPrepareLetterPage();
 
-      await userEvent.click(screen.getByRole('button', { name: 'Last ned førsteside' }));
+      await userEvent.click(screen.getByRole('button', { name: 'Last ned førsteside og søknad' }));
       expect(submitCalls).toHaveLength(1);
       const submitted = submitCalls[0].target as HTMLFormElement;
       expect((submitted.elements[0] as HTMLInputElement).name).toBe('form');
-      expect((submitted.elements[1] as HTMLInputElement).name).toBe('submissionData');
+      expect((submitted.elements[1] as HTMLInputElement).name).toBe('submission');
     });
 
     it('Laster ikke ned førsteside pdf dersom enhet ikke er valgt, og viser feilmelding i stedet', async () => {
@@ -165,7 +165,7 @@ describe('PrepareLetterPage', () => {
       });
       renderPrepareLetterPage(form);
 
-      await userEvent.click(await screen.findByRole('button', { name: 'Last ned førsteside' }));
+      await userEvent.click(await screen.findByRole('button', { name: 'Last ned førsteside og søknad' }));
       expect(submitCalls).toHaveLength(1);
       expect(await screen.findByText(TEXTS.statiske.prepareLetterPage.entityNotSelectedError)).toBeInTheDocument();
     });

@@ -182,7 +182,7 @@ const en = {
         'Rediger søknaden': { value: 'Edit application', scope: 'global' },
         'Send i posten': { value: 'Submit by mail', scope: 'global' },
         'Til digital innsending': { value: 'To digital submission', scope: 'global' },
-        'Last ned førsteside': { value: 'Download cover sheet', scope: 'global' },
+        'Last ned førsteside og søknad': { value: 'Download cover sheet and applicaton', scope: 'global' },
         'Jeg har lest instruksjonene.': { value: 'I have read the instructions.', scope: 'global' },
         Ja: { value: 'Yes', scope: 'global' },
         Nei: { value: 'No', scope: 'global' },

@@ -175,7 +175,7 @@ const nnNO = {
         'Rediger søknaden': { value: 'Rediger søknaden', scope: 'global' },
         'Send i posten': { value: 'Send i posten', scope: 'global' },
         'Til digital innsending': { value: 'Til digital innsending', scope: 'global' },
-        'Last ned førsteside': { value: 'Last ned førsteside', scope: 'global' },
+        'Last ned førsteside og søknad': { value: 'Last ned førsteside og søknad', scope: 'global' },
         'Jeg har lest instruksjonene.': { value: 'Eg har lese rettleiinga.', scope: 'global' },
         valgfritt: { value: 'valfritt', scope: 'global' },
         Januar: { value: 'Januar', scope: 'global' },

@@ -20,7 +20,7 @@ const dbEntity = {
       'Rediger søknaden': 'Edit application',
       'Send i posten': 'Submit by mail',
       'Til digital innsending': 'To digital submission',
-      'Last ned førsteside': 'Download cover sheet',
+      'Last ned førsteside og søknad': 'Download cover sheet and application',
       'Jeg har lest instruksjonene.': 'I have read the instructions.',
       Ja: 'Yes',
       Nei: 'No',

@@ -22,7 +22,7 @@ const dbEntity = {
       'Rediger søknaden': 'Rediger søknaden',
       'Send i posten': 'Send i posten',
       'Til digital innsending': 'Til digital innsending',
-      'Last ned førsteside': 'Last ned førsteside',
+      'Last ned førsteside og søknad': 'Last ned førsteside og søknad',
       'Jeg har lest instruksjonene.': 'Eg har lese rettleiinga.',
       valgfritt: 'valfritt',
       Januar: 'Januar',

@@ -63,7 +63,7 @@ export const grensesnitt = {
     editAnswers: 'Fortsett utfylling',
   },
   prepareLetterPage: {
-    downloadCoverPage: 'Last ned førsteside',
+    downloadCoverPage: 'Last ned førsteside og søknad',
   },
   stepper: {
     toggleText: 'Steg {{currentStep}} av {{totalSteps}}',