Corrige les incohérences de dates et ajoute les dates de publication et de modification de la déclaration #1331

bellangerq · 2025-12-04T11:29:04Z

Documentation
~~Migration audits existants~~ Pas besoin finalement (on affiche pas la date si on a pas la date de rédaction de la déclaration
Tests e2e

confiture-rest-api/src/audits/audit.service.ts

hissalht · 2025-12-11T16:32:13Z

confiture-rest-api/src/audits/audit.service.ts

+    const audit = await this.prisma.audit.findUnique({
+      where: { editUniqueId: uniqueId }
+    });


pas besoin de fetcher tout l’objet

Suggested change

const audit = await this.prisma.audit.findUnique({

where: { editUniqueId: uniqueId }

});

const audit = await this.prisma.audit.findUnique({

where: { editUniqueId: uniqueId },

select: { statementPublicationDate: true }

});

hissalht · 2025-12-11T16:38:11Z

confiture-rest-api/src/audits/audit.service.ts

+      where: { editUniqueId: uniqueId }
+    });
+
+    return this.prisma.audit.updateMany({


On peut utiliser prisma.audit.update() ici je pense.

hissalht · 2025-12-11T16:39:46Z

confiture-rest-api/src/audits/audit.service.ts

+      where: { editUniqueId: uniqueId }
+    });
+
+    return this.prisma.audit.updateMany({


⚠️ La promesse est retournée mais aucun appel à updateStatementDate n’utilise la valeur retournée.

Effectivement...

hissalht · 2025-12-11T16:47:19Z

confiture-rest-api/src/audits/audit.service.ts

+        this.updateStatementDate(uniqueId);
+      }
+
      return audit;


⚠️ Si la date de la décla est mise à jour, l’objet audit retourné ici ne contient pas la date mise à jour, ce qui provoque peut-être des incohérences côté front

Tu peux voir comment j’ai fait sur la PR de la mise à jour de la date ici

Ara/confiture-rest-api/src/audits/audit.service.ts

Lines 369 to 371 in 325f809

if (!changedProperties.every(changedProperty => ignoredChanges.includes(changedProperty))) {

return (await this.updateAuditEditDate(uniqueId)) ?? audit;

}

confiture-rest-api/src/audits/audit.service.ts

hissalht · 2025-12-11T17:06:24Z

confiture-web-app/src/components/audit/AuditGenerationHeader.vue

+            {{ formatDate(auditStore.currentAudit?.publicationDate) }}
+          </time>
+        </strong>
+        <span v-if="auditStore.currentAudit?.editionDate && !isSameDay(auditStore.currentAudit?.publicationDate, auditStore.currentAudit?.editionDate)" class="fr-text--xs fr-mb-0 audit-status-modify">


on avait pas fait un truc pour avoir max 1 attribut par ligne ?

Si... Je viens de l'aider en mettant un attribut à la ligne et il m'a mis l'autre sur une autre ligne en sauvegardant le fichier. Meh.

hissalht · 2025-12-12T16:09:01Z

confiture-web-app/src/pages/StatementPage.vue

-            <template v-if="report.data.updateDate">Elle a été mise à jour le
-              <strong>{{ formatDate(report.data.updateDate) }}</strong>.</template>
+          <p class="fr-mb-2w fr-mb-md-3w">
+            Cette déclaration a été établie le {{ report.data.statementPublicationDate ? formatDate(report.data.statementPublicationDate) : '[JJ/MM/YYYY]' }}.


C’est normal le [JJ/MM/YYYY] ?

AdrienMuzyczka · 2025-12-15T15:33:25Z

Suite à tests du 15/12/2025

Retours

Prendre en compte l’évaluation des critères transverses pour le commencement de l’audit. Dès que le statut d’un critère (transverse ou non) est renseigné :
- Pages « Tableau de bord » et « Mes audits » : le bouton « Commencer l’audit » devient « Continuer l’audit ».
Gérer le cas où la dernière thématique est définie comme « NA sur la page » par l’usager. Dans ce cas, afficher « Audit terminé le » au lieu de faire passer l’indicateur de progression à 100 %.
Page « Audit » : revoir l’icône « check » associée à la mention « Audit terminé le ». Cf. maquette
Page « Rédaction de la déclaration » : ajuster l’espacement entre « champ obligatoire » et « Nom du site audité » lorsqu’il n’y a pas de date de rédaction à 32 px. Cf. maquette
Ajouter la date de modification de la déclaration lorsqu’elle est modifiée.
Ajouter la date de modification de l’audit lorsqu’il est modifié.

Remarques et cas particuliers

Dates de création d’audit affichée dans la page “Mes audits” + wording bouton “Commencer l’audit” à contrôler en prod.

--> À revoir :

Pour un audit terminé :
- L'usager ajoute une nouvelle page à l'échantillon OU change de type d'audit (25 ou 50 vers 50 ou 106) :
  - la date de fin d’audit est remplacée par le taux de progression (X %) ;
  - lorsque l’audit est de nouveau terminé :
    - la date de fin d’audit correspond à la date du premier achèvement ;
    - la date de modification de l’audit correspond à la date du second achèvement.

… publication date

…mpletion

…cation

…it, statement, report)

…hboard page

hissalht temporarily deployed to ara-1255-bug-les-dates--rsc5si December 4, 2025 11:32 Inactive

hissalht temporarily deployed to ara-1255-bug-les-dates--rsc5si December 4, 2025 13:18 Inactive

hissalht temporarily deployed to ara-1255-bug-les-dates--rsc5si December 4, 2025 14:06 Inactive

hissalht temporarily deployed to ara-1255-bug-les-dates--rsc5si December 10, 2025 13:13 Inactive

hissalht temporarily deployed to ara-1255-bug-les-dates--rsc5si December 10, 2025 13:43 Inactive

bellangerq marked this pull request as ready for review December 10, 2025 13:48

bellangerq requested review from AdrienMuzyczka, benoitdequick, hissalht and yaaax December 10, 2025 13:48

hissalht temporarily deployed to ara-1255-bug-les-dates--rsc5si December 10, 2025 14:24 Inactive

hissalht temporarily deployed to ara-1255-bug-les-dates--rsc5si December 11, 2025 09:32 Inactive

benoitdequick approved these changes Dec 11, 2025

View reviewed changes

hissalht requested changes Dec 12, 2025

View reviewed changes

bellangerq added 13 commits December 17, 2025 12:06

styles: update spacings in pages headers

b859a54

wording: update existing dates wording and use correct typography dash

e8514a2

feat: add statement publication and edition dates to audit model

be52b79

feat: display statement publication and edition dates in pages

63dafd8

feat: reset statement dates when duplicating audit

931c16e

fix: update statement edition date on audit change only if there is a…

fcfe3df

… publication date

fix: spacing on statement step on overview page

5f4bc28

doc: dates and how they are triggered

1fe7d72

test: correct already failing tests

2bd4115

fix: add date placeholder if statement publication date is undefined

64cb140

feat: take into account transverse criteria when calculating audit co…

e51bb25

…mpletion

fix: dont show statement edition date if same date as statement publi…

9ba217c

…cation

fix: dont show edition date if same date as publication date (for aud…

e7f6aae

…it, statement, report)

hissalht temporarily deployed to ara-1255-bug-les-dates--rsc5si December 17, 2025 12:24 Inactive

bellangerq force-pushed the 1255-bug-les-dates-affichées-dans-la-déclaration-sont-incohérentes-ou-absentes branch from 2558d1b to e7f6aae Compare December 17, 2025 12:28

hissalht temporarily deployed to ara-1255-bug-les-dates--rsc5si December 17, 2025 12:28 Inactive

bellangerq added 3 commits December 17, 2025 17:30

fix: icon style, spacing and component lint

4206f50

fix: include transverse criteria in progress calculation on audit das…

b58b3b5

…hboard page

fix: pr feedback in audit services

bbe25a7

hissalht temporarily deployed to ara-1255-bug-les-dates--rsc5si December 17, 2025 16:58 Inactive

hissalht requested a deployment to ara-1255-bug-les-dates--rsc5si December 17, 2025 16:58 Abandoned

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Corrige les incohérences de dates et ajoute les dates de publication et de modification de la déclaration #1331

Corrige les incohérences de dates et ajoute les dates de publication et de modification de la déclaration #1331

Uh oh!

bellangerq commented Dec 4, 2025 •

edited

Loading

Uh oh!

Uh oh!

hissalht Dec 11, 2025

Uh oh!

hissalht Dec 11, 2025

Uh oh!

hissalht Dec 11, 2025

Uh oh!

bellangerq Dec 17, 2025

Uh oh!

hissalht Dec 11, 2025

Uh oh!

Uh oh!

hissalht Dec 11, 2025

Uh oh!

bellangerq Dec 17, 2025

Uh oh!

hissalht Dec 12, 2025

Uh oh!

bellangerq Dec 17, 2025

Uh oh!

AdrienMuzyczka commented Dec 15, 2025 •

edited by bellangerq

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

	if (!changedProperties.every(changedProperty => ignoredChanges.includes(changedProperty))) {
	return (await this.updateAuditEditDate(uniqueId)) ?? audit;
	}

Corrige les incohérences de dates et ajoute les dates de publication et de modification de la déclaration #1331

Are you sure you want to change the base?

Corrige les incohérences de dates et ajoute les dates de publication et de modification de la déclaration #1331

Uh oh!

Conversation

bellangerq commented Dec 4, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

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!

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!

AdrienMuzyczka commented Dec 15, 2025 • edited by bellangerq Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Suite à tests du 15/12/2025

Retours

Remarques et cas particuliers

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

bellangerq commented Dec 4, 2025 •

edited

Loading

AdrienMuzyczka commented Dec 15, 2025 •

edited by bellangerq

Loading