Skip to content

Contrôle a posteriori : Mise à jour d'une mention relative aux justificatifs [GEN-2657] - #6723

Merged
leo-naeka merged 1 commit into
masterfrom
leo/update-evaluations-mention
Aug 21, 2025
Merged

leo-naeka merged 1 commit into
masterfrom
leo/update-evaluations-mention

Conversation

@leo-naeka

@leo-naeka leo-naeka commented Aug 19, 2025

Copy link
Copy Markdown
Contributor

🤔 Pourquoi ?

Mise en conformité légale : l’Arrêté du 12/04/22 exige un document papier.

🍰 Comment ?

Mise à jour de la mention

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?
  • Ajouter l'étiquette « Bug » ?

💻 Captures d'écran

Capture d’écran 2025-08-19 à 18 58 41

@leo-naeka leo-naeka self-assigned this Aug 19, 2025
@leo-naeka leo-naeka added the modifié Modifié dans le changelog. label Aug 19, 2025
@notion-workspace

Copy link
Copy Markdown

@vincentporte vincentporte left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎯

</span>
</p>
<p class="mb-1">Ce critère administratif est certifié par l’État, aucun justificatif n’est requis.</p>
<p class="mb-1">

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hors scope : voulons-nous conserver ces assertContains alors que l'HTML est inclus aussi dans les snapshots ?

@francoisfreitag francoisfreitag Aug 20, 2025

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Les assertContains vérifient la présence de l’encart « certifié ». On pourrait peut-être éliminer un peu de markup pour les rendre plus concis, mais on devrait les garder.

@leo-naeka
leo-naeka added this pull request to the merge queue Aug 21, 2025
Merged via the queue into master with commit 994f6d0 Aug 21, 2025
15 checks passed
@leo-naeka
leo-naeka deleted the leo/update-evaluations-mention branch August 21, 2025 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

modifié Modifié dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants