feat(information): migrate to HTML content #1463

m-maillot · 2024-08-29T11:57:34Z

No description provided.

sonarcloud · 2024-08-29T11:58:06Z

Quality Gate failed

Failed conditions
81.9% Duplication on New Code (required ≤ 15%)
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

m-maillot · 2024-08-29T12:01:00Z

targets/frontend/src/components/data/Head.tsx

+  rowCount: number;
+};
+
+export const EnhancedTableHead = <T extends Data>({


Création d'un tableau générique permettant d'afficher la liste des documents, d'en sélectionner et de publier.
C'est un composant générique graphique mais avec également une partie publication qui est commune aux différents contenu. Cette partie affiche une modale avec les contenus sélectionnés et permet de les publier et de suivre l'avancement de l'opération.
Son principal objectif dans un premier temps est de nous permettre de republier des contenus après des migrations de contenu.

Par la suite, on pourra ajouter un bouton pour supprimer. Ajouter le suivi des statuts et principalement celui de savoir si le contenu est publié et dispo en prod, à publier (version en attente de publication)...

Ok top, j'aurai juste nommé le fichier composant EnhancedTableHead au lieu de Head

m-maillot · 2024-08-29T12:02:15Z

targets/frontend/src/modules/agreements/components/Edition/index.tsx

@@ -11,13 +11,12 @@ import React from "react";
 import { useAgreementUpdateMutation } from "./agreement.mutation";
 import { useAgreementQuery } from "./agreement.query";
 import { AgreementForm } from "../Common";
-import { usePublishMutation } from "./publish.mutation";
+import { usePublishMutation } from "../../../documents/components/publish.mutation";


J'ai homogénéisé le usePublicationMutation pour en avoir un seul dans documents. Ca évite de devoir modifier X graphQL quand on veut modifier l'API...

m-maillot · 2024-08-29T12:03:09Z

targets/frontend/src/modules/agreements/components/List/Filter/index.tsx

+  };
+
+  return (
+    <FormControl sx={{ width: 200, marginX: "8px" }} size="small">


Ce composant est une amélioration du filtre sur les conventions collectives. Avant on avait un switch, maintenant on a une liste à choix multiple. Plus simple à comprendre et plus jolie.

Avant :

Après :

m-maillot · 2024-08-29T12:13:01Z

targets/frontend/src/modules/agreements/components/List/index.tsx

-          color="success"
-          onClick={() => {
-            router.push("/agreements/creation");
+    <EnhancedTable<AgreementData>


Ici, on remplace le code répété pour chaque contenu qui affiche les documents dans un tableau. On remplace par le composant EnhancedTable.

Ce composant prend la source pour faire automatiquement le publish.
Il va ensuite prendre des headCells qui sont les différentes colonnes. Pour chaque colonne, on indique la donnée dans l'objet à afficher et on peut customiser le render si besoin.
On a ensuite rows qui contient le contenu du tableau donc ici les conventions collectives.
On a ensuite les onClick permettant de rediriger.
Le setSearch qui permet d'appliquer la recherche
Et pour finir le customFilters qui permet d'ajouter des filtres spécifiques (comme ici sur les CCs traitées vs non traitées).

Sympa le generic pour bien typer

m-maillot · 2024-08-29T12:13:48Z

targets/frontend/src/modules/agreements/components/List/list.query.ts

@@ -10,8 +10,7 @@ export const listAgreementsQuery = gql`
    agreements: agreement_agreements(
      order_by: { id: asc }
      where: {
-        id: { _ilike: $idcc }
-        name: { _ilike: $keyword }
+        _or: [{ id: { _ilike: $idcc } }, { shortName: { _ilike: $keyword } }]


Petite amélioration sur la recherche dans les CCs. Au lieu d'avoir deux champs, un pour filtrer sur l'idcc et un autre sur le nom, on a un seul champ et on filtre sur les deux.

m-maillot · 2024-08-29T12:14:54Z

targets/frontend/src/modules/documents/components/publish.mutation.ts

+  props: PublishProps
+) => Promise<OperationResult<Result>>;
+
+export const usePublishMutation = (): PublishMutationResult => {


Ici c'est le usePublishMutation générique que l'on réutilise ensuite. Ca permet d'avoir qu'une requête graphQL écrite pour publier dans le code (comme la fonction est générique, c'est plus simple de le faire comme ça).

m-maillot · 2024-08-29T12:15:58Z

targets/frontend/src/modules/informations/components/informationsEdit/InformationsBlockText.tsx

@@ -21,14 +21,11 @@ export const InformationsBlockText = ({
        spacing={2}
      >
        <FormControl>
-          <FormTextField
+          <FormEditionField


Ce qui nous intéresse dans cette PR :)
J'ai remplacé par le composant tiptap pour éditer une page info.

m-maillot · 2024-08-29T12:16:20Z

targets/frontend/src/modules/informations/mapInformationToDocument.ts

@@ -48,7 +48,7 @@ export const mapInformationToDocument = async (
              blocks: await Promise.all(
                blocks.map(async (block) => {
                  const htmlWithGlossary = await getGlossaryContent(
-                    "markdown",
+                    "html",


On a maintenant du html en entrée :)

m-maillot · 2024-08-29T12:16:52Z

targets/hasura/metadata/actions.yaml

@@ -4,6 +4,7 @@ actions:
      kind: synchronous
      handler: '{{API_URL}}/actions/publish'
      forward_client_headers: true
+      timeout: 180


J'ai monté le timeout du publish car en local j'avais des pages infos qui prenaient trop de temps.

m-maillot · 2024-08-29T12:17:45Z

targets/hasura/migrations/default/1723118781764_update_information_markdown_to_html/up.sql

@@ -0,0 +1,1500 @@
+UPDATE information.informations_contents_blocks


Là on migre les données en markdown en html. Il faudra vérifier qu'aucune page info n'a été mise à jour au moment de merge et release la feature.

maxgfr

top gg! j'ai juste lu le code mais pas recetté pour info

maxgfr · 2024-08-30T10:16:33Z

targets/frontend/src/components/data/Head.tsx

+  rowCount: number;
+};
+
+export const EnhancedTableHead = <T extends Data>({


Ok top, j'aurai juste nommé le fichier composant EnhancedTableHead au lieu de Head

maxgfr · 2024-08-30T10:18:16Z

targets/frontend/src/modules/agreements/components/Edition/index.tsx

@@ -11,13 +11,12 @@ import React from "react";
 import { useAgreementUpdateMutation } from "./agreement.mutation";
 import { useAgreementQuery } from "./agreement.query";
 import { AgreementForm } from "../Common";
-import { usePublishMutation } from "./publish.mutation";
+import { usePublishMutation } from "../../../documents/components/publish.mutation";


maxgfr · 2024-08-30T10:18:57Z

targets/frontend/src/modules/agreements/components/List/index.tsx

-          color="success"
-          onClick={() => {
-            router.push("/agreements/creation");
+    <EnhancedTable<AgreementData>


Sympa le generic pour bien typer

maxgfr · 2024-08-30T10:19:33Z

targets/frontend/src/modules/agreements/components/List/index.tsx

+                {value ? (
+                  <Tooltip title="Convention collective traitée par le CDTN">
+                    <GavelIcon fontSize="small" />
+                  </Tooltip>
+                ) : undefined}


Suggested change

{value ? (

<Tooltip title="Convention collective traitée par le CDTN">

<GavelIcon fontSize="small" />

</Tooltip>

) : undefined}

{value && (

<Tooltip title="Convention collective traitée par le CDTN">

<GavelIcon fontSize="small" />

</Tooltip>

)}

maxgfr · 2024-08-30T10:19:48Z

targets/frontend/src/modules/agreements/components/List/list.query.ts

@@ -10,8 +10,7 @@ export const listAgreementsQuery = gql`
    agreements: agreement_agreements(
      order_by: { id: asc }
      where: {
-        id: { _ilike: $idcc }
-        name: { _ilike: $keyword }
+        _or: [{ id: { _ilike: $idcc } }, { shortName: { _ilike: $keyword } }]


maxgfr · 2024-08-30T10:20:06Z

targets/frontend/src/modules/documents/components/publish.mutation.ts

maxgfr · 2024-08-30T10:20:54Z

targets/hasura/metadata/actions.yaml

@@ -4,6 +4,7 @@ actions:
      kind: synchronous
      handler: '{{API_URL}}/actions/publish'
      forward_client_headers: true
+      timeout: 180


feat(information): migrate to HTML content

259d726

m-maillot requested review from Viczei, carolineBda and maxgfr August 29, 2024 11:57

m-maillot commented Aug 29, 2024

View reviewed changes

maxgfr approved these changes Aug 30, 2024

View reviewed changes

Viczei approved these changes Aug 30, 2024

View reviewed changes

m-maillot mentioned this pull request Sep 18, 2024

feat: ajout d'une fonctionnalité pour publier plusieurs contenus #1471

Closed

m-maillot closed this Sep 23, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(information): migrate to HTML content #1463

feat(information): migrate to HTML content #1463

m-maillot commented Aug 29, 2024

sonarcloud bot commented Aug 29, 2024

m-maillot Aug 29, 2024

maxgfr Aug 30, 2024

m-maillot Aug 29, 2024

maxgfr Aug 30, 2024

m-maillot Aug 29, 2024

m-maillot Aug 29, 2024

m-maillot Aug 29, 2024

maxgfr Aug 30, 2024

m-maillot Aug 29, 2024

maxgfr Aug 30, 2024

m-maillot Aug 29, 2024

m-maillot Aug 29, 2024

m-maillot Aug 29, 2024

m-maillot Aug 29, 2024

maxgfr Aug 30, 2024

m-maillot Aug 29, 2024

maxgfr left a comment

maxgfr Aug 30, 2024

maxgfr Aug 30, 2024

maxgfr Aug 30, 2024

maxgfr Aug 30, 2024

maxgfr Aug 30, 2024

maxgfr Aug 30, 2024

maxgfr Aug 30, 2024

		@@ -0,0 +1,1500 @@
		UPDATE information.informations_contents_blocks

feat(information): migrate to HTML content #1463

feat(information): migrate to HTML content #1463

Conversation

m-maillot commented Aug 29, 2024

sonarcloud bot commented Aug 29, 2024

Quality Gate failed

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

maxgfr left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment