feat: ajouter une nouvelle contribution #1524

m-maillot · 2025-01-09T07:55:19Z

No description provided.

m-maillot · 2025-01-09T07:56:17Z

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

@@ -11,6 +11,12 @@ export const generateMetadata = (
  contribution: DocumentElasticWithSource<ContributionDocumentJson>,
  breadcrumbs: Breadcrumb[]
 ): ContributionMetadata => {
+  if (breadcrumbs.length === 0) {


J'ai remonté la condition pour le valider même si c'est une contribution générique. Sinon, on pouvait avoir une contrib générique sans thème et ça fait planter le front.
J'en ai profité pour revoir le message d'erreur.

m-maillot · 2025-01-09T07:56:54Z

targets/frontend/src/components/contributions/questions/common/Form.tsx

 });

 export type QuestionFormData = z.infer<typeof formDataSchema>;

 export const Form = ({
  question,
  messages,
-  onUpsert,
+  defaultOrder,


Je passe un defaultOrder pour renseigné par défaut le prochain index disponible en BDD

m-maillot · 2025-01-09T07:57:15Z

targets/frontend/src/components/contributions/questions/common/Form.tsx

+            }
+            label="Ordre"
+            fullWidth
+            disabled={question !== undefined}


Je désactive si on est en mode édition.

m-maillot · 2025-01-09T07:59:50Z

targets/frontend/src/components/contributions/questions/creation/question.query.ts

+    }
+    maxOrder: contribution_questions(order_by: { order: desc }, limit: 1) {
+      order
+    }


Ici je récupère les messages blocks, la liste des CCs supportées (j'ai besoin de savoir si elles sont unextended pour une règle métier que je gère plus tard) et le maxOrder qui me permet d'avoir le prochain order disponible et de le renseigner par défaut dans le formulaire de création

m-maillot · 2025-01-09T08:00:51Z

targets/frontend/src/components/contributions/questions/creation/question.query.ts

+      messages: data.messages,
+      agreementIds: data.agreements
+        .flatMap((item) => item)
+        .concat([{ id: "0000", unextended: false }]),


Ici j'ajoute pour générer la réponse à la générique. Je ne sais pas si c'est pas mieux de le faire dans le code de création d'une nouvelle contrib.

ça me choque pas ici

m-maillot · 2025-01-09T08:01:46Z

targets/frontend/src/components/contributions/type.ts

@@ -154,7 +154,7 @@ export const questionBaseSchema = z.object({
      required_error: "Une question doit être renseignée",
    })
    .min(1, "Une question doit être renseignée"),
-  order: z.number(),
+  order: z.coerce.number(),


Cela permet d'obliger zod à convertir en number la valeur avant de la valider. C'est pour faire fonctionner avec le formulaire car l'input retourne un string.

m-maillot · 2025-01-09T08:08:03Z

targets/frontend/src/modules/documents/api/documents.service.ts

+        if (!contribution.cdtnId) {
+          postTreatment = async (document) => {
+            await this.contributionRepository.updateCdtnId(
+              contribution.id,
+              document.cdtn_id
+            );
+          };


Ici ça corrige le soucis des états de publication qui n'étaient pas bon. Avant, on tentait de mettre à jour le cdtn_id à ce moment mais le document n'est pas encore créé. Du coup, on a une erreur hasura de foreign key. J'ai placé le processus dans un postTreatment que l'on va exécuté à la fin après avoir insérer le document.

m-maillot · 2025-01-09T08:12:36Z

targets/frontend/src/components/contributions/questions/creation/index.tsx

+        data: data.agreementIds.map((item) => ({
+          display_date: "01/01/2025",
+          agreement_id: item.id,
+          content_type: item.unextended ? "NOTHING" : undefined,
+        })),


On va créer les questions ici. Du coup, on utilise la liste des cc que l'on a récupéré auparavant.

La ligne content_type: item.unextended ? "NOTHING" : undefined permet de corriger le soucis du champ non éditable sur certaines réponses. En fait, pour les CCs non étendues (comme la 29), on désactive ce champ car elle ne peut être que dans l'état UNKNOWN pour indiquer la cc est non étendue et que l'on a pas le choix que de mettre une réponse à l'utilisateur. C'est géré dans le formulaire d'une réponse directement.

m-maillot · 2025-01-09T08:14:13Z

targets/frontend/src/components/contributions/questions/creation/index.tsx

+      message_id: formData.message_id ? formData.message_id : undefined, // use to transform empty string sent by the form to undefined
+      answers: {
+        data: data.agreementIds.map((item) => ({
+          display_date: "01/01/2025",


A voir par la suite :

Ce champ n'est pas nullable actuellement. Mais c'est étrange de mettre une date à ce moment. Est ce que l'on met la date de création de la contribution ? Est ce que l'on ne met rien ? Il faudrait ajouter le message d'alerte quand on sauvegarde une réponse pour rappeler qu'il faut mettre à jour la date de mise à jour. Car il ça risque d'être oublier je pense.

il faudrait mettre la date de création de la question. Une date en dur c'est un peu bizarre.

Le soucis c'est qu'elle ne devrait pas être renseigné avant la publication car elle va changer :(

m-maillot · 2025-01-13T10:38:06Z

.kontinuous/env/dev/values.yaml

+    annotations:
+      oblik.socialgouv.io/min-request-cpu: "500m"
+      oblik.socialgouv.io/min-request-memory: "500Mi"
+      oblik.socialgouv.io/min-limit-cpu: "1000m"
+      oblik.socialgouv.io/min-limit-memory: "1Gi"


On force kube à nous filer plus de ressources sinon la mise à jour des docs échouent.

m-maillot · 2025-01-13T10:38:57Z

.kontinuous/values.yaml

@@ -98,7 +98,7 @@ export:

 hasura:
  ~chart: hasura
-  ~needs: [build-hasura, pg]
+  ~needs: [ build-hasura, pg ]


Il manquait la dépendance sur pg et hasura se lançait avant pg et du coup ça plante le process de déploiement.

carolineBda · 2025-01-13T14:14:12Z

.kontinuous/env/dev/values.yaml

@@ -59,3 +59,8 @@ pg:
          region:
            ~tpl~name: "cdtn-prod-backups-access-key"
            key: bucket_region
+    annotations:


ça on commit du coup ?

Je dirai que oui pour ne pas être bloqué sur les branches de dev...

carolineBda · 2025-01-13T14:17:50Z

targets/frontend/src/components/contributions/questions/common/Form.tsx

+            control={control}
+            hintText={
+              <>
+                Cette valeur est indicative. Vous pouvez modifier cette valeur


Suggested change

Cette valeur est indicative. Vous pouvez modifier cette valeur

Cette valeur est indicative. Vous pouvez la modifier mais le numéro de question ne doit pas déjà être utilisée.

carolineBda · 2025-01-13T14:18:03Z

targets/frontend/src/components/contributions/questions/common/Form.tsx

+              <>
+                Cette valeur est indicative. Vous pouvez modifier cette valeur
+                mais elle ne doit pas déjà être utilisée.{" "}
+                <b>


Suggested change

<b>

<strong>

carolineBda · 2025-01-13T14:23:11Z

targets/frontend/src/pages/contributions/questions/creation.tsx

+
+export function ContributionsPage() {
+  return (
+    <Layout title="Contributions">


Suggested change

<Layout title="Contributions">

<Layout title="Nouvelle Contribution">

m-maillot · 2025-01-20T14:20:42Z

targets/frontend/src/components/contributions/questions/creation/index.tsx

+      `/contributions/questions/${result.insert_contribution_questions_one.id}`
+    );
+  };
+  if (!data) {


Lorsque je clique sur "créer" j'ai un message furtif d'erreur qui apparait. Cela n'est pas bloquant mais cela n'est pas un comportement normal attendu.

sonarqubecloud · 2025-01-20T15:04:50Z

Quality Gate passed

Issues
4 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

tokenbureau · 2025-01-20T15:12:45Z

🎉 Deployment for commit 56ce9db :

Ingresses

🚀 https://www-cdtn-admin-linked-feat-add-contribution-37b20hcb.ovh.fabrique.social.gouv.fr/

Docker images

📦 docker pull harbor.fabrique.social.gouv.fr/cdtn/cdtn-admin/export:sha-56ce9dbcf07831826e36eaf0745eb24f51d63f7e
📦 docker pull harbor.fabrique.social.gouv.fr/cdtn/cdtn-admin/frontend:sha-56ce9dbcf07831826e36eaf0745eb24f51d63f7e
📦 docker pull harbor.fabrique.social.gouv.fr/cdtn/cdtn-admin/hasura:sha-56ce9dbcf07831826e36eaf0745eb24f51d63f7e

Debug

m-maillot commented Jan 9, 2025

View reviewed changes

m-maillot force-pushed the linked/feat/add-contribution branch from 1c475cc to 8901636 Compare January 9, 2025 08:08

m-maillot had a problem deploying to review-auto January 9, 2025 08:08 — with GitHub Actions Failure

m-maillot commented Jan 9, 2025

View reviewed changes

m-maillot requested review from maxgfr, carolineBda and Viczei January 9, 2025 08:14

m-maillot force-pushed the linked/feat/add-contribution branch from 8901636 to b682378 Compare January 9, 2025 11:53

m-maillot temporarily deployed to review-auto January 9, 2025 11:53 — with GitHub Actions Inactive

m-maillot force-pushed the linked/feat/add-contribution branch from b682378 to b52a000 Compare January 9, 2025 12:14

m-maillot had a problem deploying to review-auto January 9, 2025 12:14 — with GitHub Actions Failure

m-maillot had a problem deploying to review-auto January 9, 2025 13:34 — with GitHub Actions Failure

m-maillot temporarily deployed to review-auto January 9, 2025 13:41 — with GitHub Actions Inactive

m-maillot temporarily deployed to review-auto January 13, 2025 10:37 — with GitHub Actions Inactive

m-maillot commented Jan 13, 2025

View reviewed changes

m-maillot force-pushed the linked/feat/add-contribution branch from fc99402 to e76a7c4 Compare January 13, 2025 11:02

m-maillot temporarily deployed to review-auto January 13, 2025 11:02 — with GitHub Actions Inactive

m-maillot temporarily deployed to review-auto January 13, 2025 13:46 — with GitHub Actions Inactive

carolineBda reviewed Jan 13, 2025

View reviewed changes

carolineBda approved these changes Jan 13, 2025

View reviewed changes

Viczei approved these changes Jan 13, 2025

View reviewed changes

maxgfr removed their request for review January 16, 2025 16:24

m-maillot added 3 commits January 20, 2025 14:08

feat: ajouter une nouvelle contribution

79dcc83

fix: increase pg ressources

79820d6

feat: ajout d'un bouton créer comme sur les autres pages

3bbb299

m-maillot commented Jan 20, 2025

View reviewed changes

m-maillot force-pushed the linked/feat/add-contribution branch from 5d97fdb to 21c96ec Compare January 20, 2025 14:30

m-maillot temporarily deployed to review-auto January 20, 2025 14:30 — with GitHub Actions Inactive

fix: feedbacks

56ce9db

m-maillot force-pushed the linked/feat/add-contribution branch from 21c96ec to 56ce9db Compare January 20, 2025 15:04

m-maillot temporarily deployed to review-auto January 20, 2025 15:04 — with GitHub Actions Inactive

m-maillot merged commit ec323b9 into master Jan 20, 2025
26 checks passed

m-maillot deleted the linked/feat/add-contribution branch January 20, 2025 15:58

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: ajouter une nouvelle contribution #1524

feat: ajouter une nouvelle contribution #1524

m-maillot commented Jan 9, 2025

m-maillot Jan 9, 2025

m-maillot Jan 9, 2025

m-maillot Jan 9, 2025

m-maillot Jan 9, 2025

m-maillot Jan 9, 2025

carolineBda Jan 13, 2025

m-maillot Jan 9, 2025

m-maillot Jan 9, 2025

m-maillot Jan 9, 2025

m-maillot Jan 9, 2025

carolineBda Jan 13, 2025

m-maillot Jan 13, 2025

m-maillot Jan 13, 2025

m-maillot Jan 13, 2025

carolineBda Jan 13, 2025

m-maillot Jan 13, 2025

carolineBda Jan 13, 2025

carolineBda Jan 13, 2025

carolineBda Jan 13, 2025

m-maillot Jan 20, 2025

sonarqubecloud bot commented Jan 20, 2025

tokenbureau bot commented Jan 20, 2025

	Cette valeur est indicative. Vous pouvez modifier cette valeur
	Cette valeur est indicative. Vous pouvez la modifier mais le numéro de question ne doit pas déjà être utilisée.

	<Layout title="Contributions">
	<Layout title="Nouvelle Contribution">

feat: ajouter une nouvelle contribution #1524

feat: ajouter une nouvelle contribution #1524

Conversation

m-maillot commented Jan 9, 2025

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

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

sonarqubecloud bot commented Jan 20, 2025

Quality Gate passed

tokenbureau bot commented Jan 20, 2025