feat(contributions): sauvegarde de la date de mise à jour des contribs

shared/types/src/elastic/contributions.ts

@@ -68,7 +68,6 @@ export type ContributionElasticDocument = ContributionElasticDocumentBase &
 type ElasticSearchContributionFicheSp = {
   type: "fiche-sp";
   url: string;
-  date: string;
   raw: string;
 };
 
@@ -104,6 +103,7 @@ type ElasticSearchContributionBase = {
   references: ContributionRef[];
   idcc: string;
   messageBlock?: string;
+  date: string;
 } & (
   | ElasticSearchContributionFicheSp
   | ElasticSearchContributionContent

shared/types/src/hasura/contributions.ts

@@ -93,6 +93,7 @@ type ContributionDocumentJsonBasic = {
   questionId: string;
   description: string;
   idcc: string; // 0000 pour la générique, impossible d'être nulle
+  date: string;
 };
 
 export type ContributionDocumentJsonContent = ContributionDocumentJsonBasic & {

targets/export-elasticsearch/src/ingester/contributions/__tests__/filterContributionToPublish.test.ts

@@ -40,6 +40,7 @@ const contributionMock: ContributionsAnswers = {
 };
 
 const doc: DocumentElasticWithSource<ContributionDocumentJson> = {
+  date: "",
   id: "",
   title: "",
   excludeFromSearch: true,

targets/export-elasticsearch/src/ingester/contributions/__tests__/generateMessageBlock.test.ts

@@ -10,7 +10,9 @@ import { generateMessageBlock } from "../generateMessageBlock";
 jest.mock("../fetchMessageBlock");
 jest.mock("../fetchAgreementMessage");
 
-const contributionGeneric: DocumentElasticWithSource<ContributionDocumentJson> | undefined = undefined;
+const contributionGeneric:
+  | DocumentElasticWithSource<ContributionDocumentJson>
+  | undefined = undefined;
 const mockContribution: any = {
   questionId: "123",
   contentType: "ANSWER",
@@ -122,14 +124,15 @@ describe("generateMessageBlock", () => {
     expect(fetchAgreementMessage).toHaveBeenCalledWith("1234");
     expect(result).toEqual("fetchedAgreementMessage");
   });
-
-
-
 });
+
 describe("Tests avec une contribution generic no cdt", () => {
-  let mockedContributionGeneric: DocumentElasticWithSource<ContributionDocumentJson> | undefined = undefined;
+  let mockedContributionGeneric:
+    | DocumentElasticWithSource<ContributionDocumentJson>
+    | undefined = undefined;
   beforeEach(() => {
     mockedContributionGeneric = {
+      date: "",
       contentType: "GENERIC_NO_CDT",
       id: "id",
       cdtnId: "cdtnId",
@@ -151,34 +154,42 @@ describe("Tests avec une contribution generic no cdt", () => {
       idcc: "0000",
       type: "generic-no-cdt",
       messageBlockGenericNoCDT: "messageBlockGenericNoCDT",
-      messageBlockGenericNoCDTUnextendedCC: "messageBlockGenericNoCDTUnextendedCC"
+      messageBlockGenericNoCDTUnextendedCC:
+        "messageBlockGenericNoCDTUnextendedCC",
     };
     (fetchMessageBlock as jest.Mock).mockResolvedValue({
       contentAgreementWithoutLegal: "agrement without legal",
-      contentNotHandledWithoutLegal: "content not handled without legal"
+      contentNotHandledWithoutLegal: "content not handled without legal",
     });
     (fetchAgreementMessage as jest.Mock).mockResolvedValue(undefined);
-  })
-  it.each(["ANSWER", "SP"])("should throw a contentAgreementWithoutLegal", async (contentType) => {
-
-    mockContribution.contentType = contentType;
-    mockContribution.idcc = "1234";
-
-    const messageBloc = await generateMessageBlock(mockedContributionGeneric, mockContribution);
-
-    expect(
-      messageBloc
-    ).toEqual("agrement without legal");
   });
+  it.each(["ANSWER", "SP"])(
+    "should throw a contentAgreementWithoutLegal",
+    async (contentType) => {
+      mockContribution.contentType = contentType;
+      mockContribution.idcc = "1234";
+
+      const messageBloc = await generateMessageBlock(
+        mockedContributionGeneric,
+        mockContribution
+      );
 
-  it.each(["NOTHING", "CDT", "UNFAVOURABLE"])("should throw a contentNotHandledWithoutLegal", async (contentType) => {
-    mockContribution.contentType = contentType;
-    mockContribution.idcc = "1234";
+      expect(messageBloc).toEqual("agrement without legal");
+    }
+  );
 
-    const messageBloc = await generateMessageBlock(mockedContributionGeneric, mockContribution);
+  it.each(["NOTHING", "CDT", "UNFAVOURABLE"])(
+    "should throw a contentNotHandledWithoutLegal",
+    async (contentType) => {
+      mockContribution.contentType = contentType;
+      mockContribution.idcc = "1234";
 
-    expect(
-      messageBloc
-    ).toEqual("content not handled without legal");
-  });
-})
+      const messageBloc = await generateMessageBlock(
+        mockedContributionGeneric,
+        mockContribution
+      );
+
+      expect(messageBloc).toEqual("content not handled without legal");
+    }
+  );
+});

targets/export-elasticsearch/src/ingester/contributions/fetchFicheSp.ts

@@ -39,7 +39,7 @@ export async function fetchFicheSp(
   if (res.error) {
     throw res.error;
   }
-  if (!res.data || res.error) {
+  if (!res.data) {
     throw new Error(`Impossible de récupérer la fiche sp ${ficheSpId}`);
   }
   if (res.data.documents.length !== 1) {

targets/frontend/src/components/contributions/answers/Comment.tsx

@@ -61,7 +61,7 @@ export const Comment = ({ comment, onDelete }: Props) => {
           <Delete fontSize="inherit" />
         </IconButton>
       </Box>
-      <Box sx={{ marginTop: 1, whiteSpace: "pre-line" }}>{comment.content}</Box>
+      <Typography sx={{ fontSize: "small" }}>{comment.content}</Typography>
     </Box>
   ) : (
     <Stack direction="row" justifyContent="space-between">

targets/frontend/src/components/contributions/answers/__tests__/AnswerForm.test.tsx

@@ -146,7 +146,7 @@ const answerBase: AnswerWithStatus = {
     export: {
       createdAt: "29/09/2023",
     },
-  },
+  }
 };
 
 const onSubmit = jest.fn(() => Promise.resolve());

targets/frontend/src/components/contributions/answers/answer.query.ts

@@ -94,6 +94,7 @@ type QueryProps = {
 export type AnswerWithStatus = Answer & {
   status: AnswerStatus;
   updateDate?: string;
+  updatedAt: string;
 };
 
 type QueryResult = {

targets/frontend/src/components/contributions/questionList/QuestionList.tsx

@@ -8,7 +8,6 @@ import {
   TableHead,
   TableRow,
   TextField,
-  Tooltip,
 } from "@mui/material";
 import { useState } from "react";
 
@@ -17,7 +16,6 @@ import {
   useQuestionListQuery,
 } from "./QuestionList.query";
 import { QuestionRow } from "./QuestionRow";
-import { statusesMapping } from "../status/data";
 
 export const countAnswersWithStatus = (
   answers: QueryQuestionAnswer[] | undefined,

targets/frontend/src/components/contributions/status/StatusPublication.tsx

@@ -25,7 +25,7 @@ export const StatusPublicationContainer = ({
     status = "PUBLISHED";
     tooltipText = `${statusesMapping[status].text} le ${format(
       parseISO(exportDate),
-      "dd/MM/yyyy HH:mm:ss"
+      "dd/MM/yyyy HH:mm"
     )}`;
   }
   return (

targets/frontend/src/components/editorialContent/ReferenceBlocks.tsx

@@ -7,7 +7,7 @@ import { FormErrorMessage } from "../forms/ErrorMessage";
 import { References } from "./References";
 import { FormRadioGroup } from "../forms";
 import React from "react";
-import { Box, Card, CardContent, Stack } from "@mui/material";
+import { Box, Card, CardContent } from "@mui/material";
 import { theme } from "src/theme";
 
 const JURIDIQUES_LABEL = "Références juridiques";

targets/frontend/src/hooks/index.js

@@ -1,21 +1,7 @@
-import { useCallback, useEffect, useRef, useState } from "react";
+import { useCallback, useState } from "react";
 
 import { debounce } from "../lib/debounce";
 
-export function usePrevious(value) {
-  // The ref object is a generic container whose current property is mutable ...
-  // ... and can hold any value, similar to an instance property on a class
-  const ref = useRef();
-
-  // Store current value in ref
-  useEffect(() => {
-    ref.current = value;
-  }, [value]); // Only re-run if value changes
-
-  // Return previous value (happens before update in useEffect above)
-  return ref.current;
-}
-
 export function useDebouncedState(defaultState, delay) {
   const [state, setState] = useState(defaultState);
   //eslint-disable-next-line react-hooks/exhaustive-deps

targets/frontend/src/modules/contribution/__tests__/mapContributionToDocument.test.ts

@@ -10,6 +10,8 @@ jest.mock("../../common/getGlossaryContent.ts", () => {
     getGlossaryContent: jest.fn(() => "mocked-glossary-content"),
   };
 });
+const updatedAt =
+  "Thu Jul 11 2024 15:18:08 GMT+0200 (Central European Summer Time)";
 
 describe("mapContributionToDocument", () => {
   const inputDoc: HasuraDocument<any> = {
@@ -31,7 +33,7 @@ describe("mapContributionToDocument", () => {
 
   it("devrait mapper l'answer d'un document sans fiche SP", async () => {
     const inputContribution: ContributionsAnswers = {
-      updatedAt: "",
+      updatedAt: updatedAt,
       id: "effee3b9-84fb-4667-944b-4b1e1fd14eb5",
       content:
         "<p>Quand une femme tombe enceinte et décide de partir en congé maternité, cette dernière a droit à des indemnités journalières de sécurité sociale venant indemniser la période durant laquelle elle ne peut plus travailler. Certaines conventions collectives prévoient également un maintien de salaire versé par l’employeur. Si le maintien est à 100%, dans ce cas, les deux mécanismes ne sont pas cumulables. Si le maintien est inférieur à 100%, le pourcentage de rémunération restant est indemnisé par les indemnités de Sécurité sociale.&nbsp;</p><h3>Maintien de salaire</h3><p>Les salariées ayant au moins une année de présence continue dans l'entreprise à la date de l'accouchement ont droit à un maintien de salaire, après déduction des indemnités de Sécurité sociale, qui leur assure leur salaire habituel, et ce pendant une durée de 36 jours (en principe 18 jours avant l’accouchement, 18 jours après).</p><p>Pour les salariées cadres âgées de moins de 25 ans et les autres salariées âgées de moins de 22 ans à la date de l'accouchement, la période de 36 jours est augmentée de 2 jours par enfant à charge. L'indemnité&nbsp;complémentaire ne pourra pas être versée plus de 46 jours. Est considéré comme enfant à charge tout enfant à charge de la salariée au sens de la législation des prestations familiales et âgé de moins de 15 ans à la date de l'accouchement.</p><p>A noter&nbsp;: Les périodes de&nbsp;suspension du contrat de travail&nbsp;(maladie, etc.) sont prises en compte pour l'ancienneté.</p><p>Si la salariée ne respecte pas la condition d’ancienneté, elle n’a pas droit au maintien de salaire versé par l’employeur mais aura potentiellement droit aux indemnités journalières de Sécurité sociale si elle respecte ses conditions d’octroi.&nbsp;</p><h3>Indemnités de Sécurité sociale</h3><p><strong>Conditions d’ouverture des droits aux indemnités journalières de Sécurité sociale</strong></p><p>Pour être indemnisée, la salariée doit remplir les conditions suivantes :</p><ul><li><p>Etre affiliée à la Sécurité sociale depuis au moins 10 mois à la date présumée de l'accouchement&nbsp;;</p></li><li><p>cesser son activité professionnelle pendant au moins 8&nbsp;semaines&nbsp;;</p></li></ul><ul><li><p>avoir :&nbsp;</p><ul><li><p>soit travaillé au moins 150 heures au cours des 3 mois civils ou des 90 jours précédant l'arrêt,&nbsp;</p></li><li><p>soit travaillé au moins&nbsp;<strong>600 heures</strong>&nbsp;au cours des 12 mois précédant l’arrêt de travail,&nbsp;</p></li><li><p>soit cotisé, au cours des 6 mois civils précédant l'arrêt, sur la base d'une rémunération au moins égale à 1 015 fois le montant du Smic horaire fixé au début de cette période,&nbsp;</p></li><li><p>soit cotisé au cours des&nbsp;<strong>12 mois</strong>&nbsp;civils précédant l’arrêt, sur la base d'une rémunération au moins égale à&nbsp;2030 fois le montant du Smic horaire fixé en début de période.</p></li></ul></li></ul><p>Exemple&nbsp;:&nbsp;le congé a débuté le 1er juillet 2023 pour une date présumée d'accouchement au 1er septembre 2023.</p><p>Le droit aux&nbsp;indemnités&nbsp;journalières est ouvert si :</p><ul><li><p>La salariée était déjà affiliée à la Sécurité sociale avant novembre 2022&nbsp;;</p></li><li><p>et a travaillé soit au moins 150 heures entre le 1er avril 2023 et le 30 juin 2023, soit au moins 600 heures entre le 1er juillet 2022 et le 30 juin 2023, soit a cotisé entre le 1er janvier 2023 et le 30 juin 2023 sur la base d'une rémunération au moins égale à 11&nbsp;439,05&nbsp;€, soit a cotisé entre le 1er juillet 2022 et le 30 juin 2023 sur la base d’une rémunération au moins égale à 22&nbsp;878,1 €.</p></li></ul><p><strong>Montant</strong></p><p>La CPAM verse des&nbsp;indemnités&nbsp;journalières, dont le montant est fixé selon les étapes de calcul suivantes :</p><ul><li><p>Calcul du salaire journalier de base : somme des 3 derniers salaires bruts perçus avant la date d'interruption du travail, divisé par 91,25.</p></li><li><p>Montant maximal du salaire journalier de base : le salaire pris en compte ne peut pas dépasser le plafond mensuel de la sécurité sociale en&nbsp;vigueur&nbsp;lors du dernier jour du mois qui précède l'arrêt (soit 3&nbsp;666&nbsp;€ par mois en 2023, ou 3&nbsp;428&nbsp;€ en 2022).</p></li><li><p>Taux forfaitaire appliqué par la CPAM : la CPAM retire à ce salaire journalier de base un taux forfaitaire de 21&nbsp;%.</p></li><li><p>Montant minimal et montant maximal des&nbsp;indemnités&nbsp;journalières : le montant ne peut pas être inférieur à 10,24&nbsp;€ ni supérieur à 95,22&nbsp;€ par jour.</p></li></ul><p><strong>Versement</strong></p><p>Les&nbsp;indemnités&nbsp;journalières sont versées tous les 14 jours.</p>",
@@ -213,6 +215,7 @@ describe("mapContributionToDocument", () => {
           },
         ],
         type: "content",
+        date: "11/07/2024",
       },
       initial_id: "effee3b9-84fb-4667-944b-4b1e1fd14eb5",
       is_available: true,
@@ -271,7 +274,7 @@ describe("mapContributionToDocument", () => {
   describe("avec une contrib generic de type GENERIC_NO_CDT", () => {
     it("devrait mapper l'answer de la contrib generic", async () => {
       const inputContribution: ContributionsAnswers = {
-        updatedAt: "",
+        updatedAt: updatedAt,
         id: "effee3b9-84fb-4667-944b-4b1e1fd14eb5",
         content: null,
         description: null,
@@ -305,6 +308,7 @@ describe("mapContributionToDocument", () => {
       );
       expect(result?.document).toEqual({
         contentType: "GENERIC_NO_CDT",
+        date: "11/07/2024",
         idcc: "0000",
         questionId: "3384f257-e319-46d1-a4cb-8e8294da337b",
         questionName:
@@ -371,6 +375,7 @@ describe("mapContributionToDocument", () => {
       // @ts-ignore
       const inputContribution: ContributionsAnswers = {
         id: "effee3b9-84fb-4667-944b-4b1e1fd14eb5",
+        updatedAt: updatedAt,
         content: "<p>Texte de la réponse</p>",
         description: "Texte de la réponse",
         content_type: "ANSWER",
@@ -422,6 +427,7 @@ describe("mapContributionToDocument", () => {
       // @ts-ignore
       const inputContribution: ContributionsAnswers = {
         id: "effee3b9-84fb-4667-944b-4b1e1fd14eb5",
+        updatedAt: updatedAt,
         content: "<p>Texte de la réponse</p>",
         description: "Texte de la réponse",
         content_type: "ANSWER",
@@ -456,6 +462,7 @@ describe("mapContributionToDocument", () => {
       // @ts-ignore
       const inputContribution: ContributionsAnswers = {
         id: "effee3b9-84fb-4667-944b-4b1e1fd14eb5",
+        updatedAt: updatedAt,
         content: "<p>Texte de la réponse</p>",
         description: "Texte de la réponse",
         content_type: "ANSWER",
@@ -507,6 +514,7 @@ describe("mapContributionToDocument", () => {
       // @ts-ignore
       const inputContribution: ContributionsAnswers = {
         id: "effee3b9-84fb-4667-944b-4b1e1fd14eb5",
+        updatedAt: updatedAt,
         content: "<p>Texte de la réponse</p>",
         description: "Texte de la réponse",
         content_type: "ANSWER",
@@ -539,6 +547,7 @@ describe("mapContributionToDocument", () => {
       // @ts-ignore
       const inputContribution2: ContributionsAnswers = {
         id: "effee3b9-84fb-4667-944b-4b1e1fd14eb6",
+        updatedAt: updatedAt,
         content: "<p>Texte de la réponse</p>",
         description: "Texte de la réponse",
         content_type: "ANSWER",

targets/frontend/src/modules/contribution/api/query.ts

@@ -7,6 +7,7 @@ export const getContributionAnswerById = gql`
       content
       description
       content_type
+      updatedAt: updated_at
       agreement {
         id
         name

targets/frontend/src/modules/contribution/mapContributionToDocument.ts

@@ -8,6 +8,7 @@ import { getReferences } from "./getReferences";
 import { generateCdtnId } from "@shared/utils";
 import { generateContributionSlug } from "./generateSlug";
 import { getGlossaryContent } from "../common/getGlossaryContent";
+import { format } from "date-fns";
 
 async function getBaseDocument(
   data: ContributionsAnswers,
@@ -77,6 +78,7 @@ export const mapContributionToDocument = async (
 
   const initalDoc: ContributionDocumentJson = {
     ...baseDoc,
+    date: format(new Date(data.updatedAt), "dd/MM/yyyy"),
     contentType: data.content_type,
     linkedContent: data.cdtn_references.map((v) => ({
       cdtnId: v.cdtn_id!,

targets/frontend/src/pages/api/sitemap.ts

@@ -81,14 +81,10 @@ export default async function Sitemap(
 
   res.setHeader("Content-Type", "text/xml");
   res.write(`<?xml version="1.0" encoding="UTF-8"?>
-  <urlset xmlns="http://www.sitemaps.org/schemas/sitemap/0.9">
-    <url>
-      <loc>${baseUrl}/</loc>
-      <lastmod>${new Date(latestPost).toISOString()}</lastmod>
-      <priority>0.8</priority>
-    </url>
-    ${pages.concat(staticPages, glossaryPages).join("")}
-  </urlset>
+<urlset xmlns="http://www.sitemaps.org/schemas/sitemap/0.9">
+<url><loc>${baseUrl}/</loc><lastmod>${new Date(latestPost).toISOString()}</lastmod><priority>0.8</priority></url>
+${pages.concat(staticPages, glossaryPages).join("")}
+</urlset>
 `);
   res.end();
   const endProcess = process.hrtime(startProcessAt);