Améliore les fonctions de débug by bellangerq · Pull Request #1457 · DISIC/Ara

bellangerq · 2026-03-26T23:25:54Z

To do / questions :

Est-ce qu'on limite ça à certains environnements (branches de review + dev ? Production aussi ?) ? Si oui, ajouter DATABASE_SEEDS en prod.

yaaax

Super fonctionnalité !

yaaax · 2026-04-01T15:08:06Z

confiture-rest-api/prisma/seed.ts

+const prisma = new PrismaClient({ adapter });
+
+async function generateAccounts() {
+  if (!process.env.DATABASE_SEEDS) {


On pourrait rajouter DATABASE_SEEDS dans le .env.example ?

Mmh on peut mais c'est qqch qu'on veut en local ?

yaaax · 2026-04-01T15:08:51Z

confiture-rest-api/prisma/seed.ts

+  };
+
+  const emails = process.env.DATABASE_SEEDS.split(",");
+  const password = await hash("pouetpouetpouet", 10);


Ça va à tout le monde pouetpouetpouet 😆 ?

Vous me dites !

confiture-rest-api/src/debug/create-audit.dto.ts

yaaax · 2026-04-01T15:38:14Z

confiture-rest-api/src/debug/create-audit.dto.ts

+  isPristine: boolean;
+
+  @IsBoolean()
+  fillStatement: boolean;


Renommer ce booléen en shouldFillStatement ou needsStatementFilled ?
(ultra pinaillage)

Ah merde je trouve ça moins clair 😅. Un avis @hissalht @emma11y ?

yaaax · 2026-04-01T15:40:49Z

confiture-rest-api/src/debug/debug.controller.ts

+              {
+                name: "Accueil",
+                url: "https://example.com",
+                slug: slugify("Accueil")


Pas possible de slugify automatiquement dans le code quelque-part ?

Je remarque que "elements-transverses" a été mis directement sans slugify("Éléments transverses")

Oui c'est vrai, autant direct mettre le slug.

yaaax · 2026-04-01T15:44:47Z

confiture-rest-api/src/app.module.ts

-    // enable debug enpoints only when the DEBUG_ENDPOINTS variable is set
-    ...(process.env.DEBUG_ENDPOINTS ? [DebugController] : [])
+    // Disable debug controller for production environment
+    ...(process.env.NODE_ENV !== "production" || process.env.GENERATE_TYPES ? [DebugController] : []),


Je me pose la question : doit-on mettre GENERATE_TYPES dans .env.example ? Ou le documenter quelque-part ? (ça sort de cette review ceci-dit…)

Mmh je pense pas, c'est une variable qui est uniquement utilisée pour lancer une commande et c'est pas configurable par l'utilisateur ni ne change selon l'environnement.

yaaax · 2026-04-01T15:45:45Z

confiture-rest-api/README.md

             postgres:14
 ```

+Optionnellement, il est possible d’ajouter la variable d’environnement `DATABASE_SEEDS` pour pré-remplir la base de données avec des comptes utilisateurs dont le mot de passe est `pouetpouetpouet`. Exemple :


voir mon autre commentaire sur pouetpouetpouet (je n’ai rien contre en vrai…)

yaaax · 2026-04-01T15:52:51Z

confiture-web-app/src/components/DebugCard.vue

+
+<template>
+  <div>
+    <button class="fr-btn fr-btn--icon-right fr-icon-arrow-down-s-line fr-mb-2w" @click="showDebugPanel = !showDebugPanel">


Plutôt utiliser un accordéon ça sera mieux pour l’accessibilité (notamment aria-expanded)

Ou alors une modale ? Car je ne sais pas si le DSFR autorise à mettre des formulaires dans des accordéons…

La modale c'est un peu relou nan ?

L'accordéon pourquoi pas, j'ai essayé mais je trouvais pas ça joli. Après je peux simplement linker le bouton et lui ajouter les bons attributs (comme le dropdown : aria-haspop, aria-expanded...).

yaaax · 2026-04-01T16:28:26Z

confiture-web-app/src/components/DebugCard.vue

+    </button>
+    <div v-if="showDebugPanel" class="debug-container">
+      <section class="fr-p-2w debug-card">
+        <h2 class="fr-h4">Créer un audit</h2>


On peut supprimer ce titre et simplifier en renommant le bouton d’ouverture "Créer un audit de test" ?

(je m’attendais à ce que ce soit une légende de <fieldset> sinon… mais en vrai pas obligé)

J'avais mis un titre car je me suis dit qu'un jour on aurait peut-être d'autres options de débug. Mais je peux supprimer en attendant oui.

confiture-web-app/src/components/DebugCard.vue

feat: create audit debug creation tool

21b3961

hissalht temporarily deployed to ara-1442-ameliorer-les--lim4yg March 26, 2026 23:28 Inactive

bellangerq added 3 commits March 27, 2026 10:53

refactor: rename debug module to tests to match what it does

b85a556

feat: create debug controller with dto

3ba2397

feat: create debug store and plug in create audit api route

ebf655d

hissalht temporarily deployed to ara-1442-ameliorer-les--lim4yg March 27, 2026 10:39 Inactive

feat: add debug card on homepage

b45e9a5

hissalht temporarily deployed to ara-1442-ameliorer-les--lim4yg March 27, 2026 10:48 Inactive

fix: move dev button on top of statement form

6f2e799

hissalht temporarily deployed to ara-1442-ameliorer-les--lim4yg March 27, 2026 10:59 Inactive

hissalht had a problem deploying to ara-1442-ameliorer-les--lim4yg March 27, 2026 15:11 Failure

bellangerq force-pushed the 1442-améliorer-les-fonctionnalités-de-debugdev branch from 72409dc to fb32e31 Compare March 27, 2026 15:58

hissalht had a problem deploying to ara-1442-ameliorer-les--lim4yg March 27, 2026 15:58 Failure

bellangerq added 2 commits March 27, 2026 17:30

feat: add user accounts seeds

f7f320d

feat: run seeds on heroku postbuild

9252cbf

bellangerq force-pushed the 1442-améliorer-les-fonctionnalités-de-debugdev branch from fb32e31 to 9252cbf Compare March 27, 2026 16:35

hissalht had a problem deploying to ara-1442-ameliorer-les--lim4yg March 27, 2026 16:36 Failure

bellangerq marked this pull request as ready for review March 27, 2026 16:38

bellangerq requested review from AdrienMuzyczka, benoitdequick, emma11y, hissalht and yaaax March 27, 2026 16:38

fix: pr feedbacks

0c235f7

hissalht temporarily deployed to ara-1442-ameliorer-les--lim4yg April 1, 2026 08:57 Inactive

fix: add emails to seed in env variable

a893df7

hissalht temporarily deployed to ara-1442-ameliorer-les--lim4yg April 1, 2026 09:12 Inactive

docs: add seeds doc

32477df

hissalht temporarily deployed to ara-1442-ameliorer-les--lim4yg April 1, 2026 09:23 Inactive

clean: remove unused dev button for account creation

978b7a0

hissalht temporarily deployed to ara-1442-ameliorer-les--lim4yg April 1, 2026 09:34 Inactive

bellangerq added 2 commits April 1, 2026 15:17

feat: exclude debug things on production environment

ea444e7

chore: clean comment

377c73e

hissalht had a problem deploying to ara-1442-ameliorer-les--lim4yg April 1, 2026 13:17 Failure

chore: remove log and add swagger api tags

63e3eb3

hissalht had a problem deploying to ara-1442-ameliorer-les--lim4yg April 1, 2026 13:48 Failure

fix: always generate debug types regardless of env

9471698

hissalht temporarily deployed to ara-1442-ameliorer-les--lim4yg April 1, 2026 15:24 Inactive

fix: use heroku specific variable to check env

f990ee6

hissalht temporarily deployed to ara-1442-ameliorer-les--lim4yg April 1, 2026 15:48 Inactive

yaaax reviewed Apr 1, 2026

View reviewed changes

fix: show debug button on correct env

ceb0fc3

hissalht temporarily deployed to ara-1442-ameliorer-les--lim4yg April 2, 2026 08:36 Inactive

fix: use hostname to detect heroku

f538a74

hissalht temporarily deployed to ara-1442-ameliorer-les--lim4yg April 2, 2026 08:55 Inactive

fix: pr feedbacks

db8e3d5

hissalht deployed to ara-1442-ameliorer-les--lim4yg April 2, 2026 09:50 View deployment

Conversation

bellangerq commented Mar 26, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

yaaax left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

bellangerq commented Mar 26, 2026 •

edited

Loading