Déplace le nom de la structure dans la déclaration (#574)

* remove auditor org from first step

* add auditor organisation field on statement edit page

* display auditor organisation in a11y statement

* remove auditor org from create-audit dto

* fix typing error

* pr feedbacks

* update changelog

Author: bellangerq

confiture-rest-api/prisma/migrations/20231129145900_make_auditor_organisation_optionnal/migration.sql

@@ -0,0 +1,2 @@
+-- AlterTable
+ALTER TABLE "Audit" ALTER COLUMN "auditorOrganisation" DROP NOT NULL;

confiture-rest-api/prisma/schema.prisma

@@ -52,10 +52,10 @@ model Audit {
   auditorName              String?
   auditorEmail             String?
   showAuditorEmailInReport Boolean       @default(false)
-  auditorOrganisation      String
 
   // A11y declaration edition step
   initiator           String?
+  auditorOrganisation String?
   procedureUrl        String?
   contactName         String?
   contactEmail        String?

confiture-rest-api/src/audits/audit-report.dto.ts

@@ -113,6 +113,10 @@ class ReportContext {
    * @example "john-doe@example.com"
    */
   auditorEmail: string | null;
+  /**
+   * @example "Web Audit Services Corp."
+   */
+  auditorOrganisation: string;
 
   technologies: string[];
 

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

@@ -58,7 +58,6 @@ export class AuditService {
 
         auditorEmail: data.auditorEmail,
         auditorName: data.auditorName,
-        auditorOrganisation: data.auditorOrganisation,
 
         pages: {
           createMany: {
@@ -765,6 +764,7 @@ export class AuditService {
       context: {
         auditorName: audit.auditorName,
         auditorEmail: null,
+        auditorOrganisation: audit.auditorOrganisation,
         desktopEnvironments: audit.environments
           .filter((e) => e.platform === 'desktop')
           .map((e) => ({

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

@@ -62,10 +62,4 @@ export class CreateAuditDto {
   @IsEmail()
   @IsOptional()
   auditorEmail?: string;
-
-  /**
-   * @example "WEB AUDIT SARL"
-   */
-  @IsString()
-  auditorOrganisation: string;
 }

confiture-rest-api/src/audits/update-audit.dto.ts

@@ -86,6 +86,13 @@ export class UpdateAuditDto extends CreateAuditDto {
   @IsOptional()
   initiator?: string;
 
+  /**
+   * @example "WEB AUDIT SARL"
+   */
+  @IsString()
+  @IsOptional()
+  auditorOrganisation?: string;
+
   /**
    * @example "John Referent"
    */

confiture-web-app/src/assets/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 Tous les changements notables de Ara sont documentés ici avec leur date, leur catégorie (nouvelle fonctionnalité, correction de bug ou autre changement) et leur pull request (PR) associée.
 
+## 06/12/2023
+
+### Autres changements ⚙️
+
+- Déplace le champ du nom de la structure qui audite le site dans la page de déclaration ([#574](https://github.com/DISIC/Ara/pull/574))
+
 ## 01/12/2023
 
 ### Autres changements ⚙️

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

@@ -61,11 +61,7 @@ const procedureAuditorName = ref(
 const procedureAuditorEmail = ref(
   props.defaultValues?.auditorEmail ?? accountStore.account?.email ?? ""
 );
-const procedureAuditorOrganisation = ref(
-  props.defaultValues?.auditorOrganisation ??
-    accountStore.account?.orgName ??
-    ""
-);
+
 const pageNameFieldRefs = ref<InstanceType<typeof DsfrField>[]>([]);
 
 /**
@@ -102,9 +98,8 @@ function fillFields() {
     { name: "Accueil", url: "https://example.com" },
     { name: "Contact", url: "https://example.com/contact" },
   ];
-  procedureAuditorName.value ||= "Etienne Dupont";
-  procedureAuditorEmail.value ||= "etienne-dupont@example.com";
-  procedureAuditorOrganisation.value ||= "Example Organisation";
+  procedureAuditorName.value = "Etienne Dupont";
+  procedureAuditorEmail.value = "etienne-dupont@example.com";
 }
 
 function onSubmit() {
@@ -115,7 +110,6 @@ function onSubmit() {
     pages: pages.value.map((p) => ({ ...p, url: p.url.trim() })),
     auditorName: procedureAuditorName.value,
     auditorEmail: formatEmail(procedureAuditorEmail.value),
-    auditorOrganisation: procedureAuditorOrganisation.value,
   });
 }
 
@@ -233,15 +227,6 @@ const route = useRoute();
           hint="Sera affiché dans le rappport de l’audit pour aider le demandeur de l’audit à vous identifier s’il a des questions ou besoin d’aide."
         />
 
-        <DsfrField
-          v-if="!accountStore.account?.orgName"
-          id="procedure-auditor-organisation"
-          v-model="procedureAuditorOrganisation"
-          label="Nom de la structure"
-          hint="Sera affiché dans la déclaration d’accessibilité, cette mention est une obligation. "
-          required
-        />
-
         <DsfrField
           v-if="!accountStore.account"
           id="procedure-auditor-email"

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

@@ -1,5 +1,5 @@
 <script setup lang="ts">
-import { ref, computed } from "vue";
+import { ref } from "vue";
 import { useRoute } from "vue-router";
 
 import { useWrappedFetch } from "../composables/useWrappedFetch";
@@ -28,11 +28,6 @@ function getA11yLevel() {
 const statementRef = ref<HTMLDivElement>();
 const showCopyAlert = ref(false);
 
-const auditIsInProgress = computed(() => {
-  // The `initiator` field is requied on the a11y declaration form so we can check that it's not null
-  return !report.data?.procedureInitiator;
-});
-
 async function copyA11yStatementHTML() {
   const tagsWithSpacesRegex = /<(?<tagName>\S+)(\s+)>/g; // "<XX  >"
   const whitespaceFollowedTags = /<(?<tagName>p)>\s{1}/g; // "<p> "
@@ -241,9 +236,9 @@ function hideCopyAlert() {
         <h4 class="fr-h2">Résultats des tests</h4>
         <p class="fr-mb-9v fr-mb-md-6w">
           L’audit de conformité réalisé par
-          <strong>{{ report.data.procedureInitiator }}</strong> révèle que
-          <strong>{{ report.data.accessibilityRate }}%</strong> des critères du
-          RGAA version 4 sont respectés.
+          <strong>{{ report.data.context.auditorOrganisation }}</strong> révèle
+          que <strong>{{ report.data.accessibilityRate }}%</strong> des critères
+          du RGAA version 4 sont respectés.
         </p>
         <!--ul class="fr-mb-9v fr-mb-md-6w">
         <li>

confiture-web-app/src/components/account/settings/Profile.vue

@@ -1,5 +1,5 @@
 <script lang="ts" setup>
-import { ref, computed, watch } from "vue";
+import { ref, computed, watch, onMounted } from "vue";
 
 import { useAccountStore } from "../../../store/account";
 import { useNotifications } from "../../../composables/useNotifications";
@@ -11,6 +11,11 @@ const notify = useNotifications();
 const name = ref("");
 const orgName = ref("");
 
+onMounted(() => {
+  name.value = accountStore.account?.name || "";
+  orgName.value = accountStore.account?.orgName || "";
+});
+
 watch(accountStore, () => {
   if (!accountStore.account) {
     return;