Skip to content

1404 & 1405 permettre le partage des audits & rendre les liens des audits privés mode connecté#1450

Open
emma11y wants to merge 53 commits into
mainfrom
1404-rendre-les-liens-des-audits-privés-mode-connecté

Hidden character warning

The head ref may contain hidden characters: "1404-rendre-les-liens-des-audits-priv\u00e9s-mode-connect\u00e9"
Open

1404 & 1405 permettre le partage des audits & rendre les liens des audits privés mode connecté#1450
emma11y wants to merge 53 commits into
mainfrom
1404-rendre-les-liens-des-audits-privés-mode-connecté

Conversation

@emma11y

@emma11y emma11y commented Mar 20, 2026

Copy link
Copy Markdown
Collaborator

closes #1404 et #1405

Avant de merger la pull request, s’assurer que :

  • Les checks GitHub passent (lint...).
  • Les tests Cypress ont été lancés en local (dans le cas d’une correction de bug ou d’une nouvelle fonctionnalité).

@emma11y emma11y linked an issue Mar 20, 2026 that may be closed by this pull request
2 tasks
@emma11y emma11y marked this pull request as ready for review March 20, 2026 16:27
@bellangerq bellangerq mentioned this pull request Mar 20, 2026
2 tasks
@hissalht hissalht temporarily deployed to ara-1404-rendre-les-lie-nca2kw March 20, 2026 16:29 Inactive
@emma11y emma11y changed the title 1404 rendre les liens des audits privés mode connecté 1404 & 1405 permettre le partage des audits & rendre les liens des audits privés mode connecté Mar 20, 2026
@AdrienMuzyczka

AdrienMuzyczka commented Mar 23, 2026

Copy link
Copy Markdown
Collaborator
  • Pages "Audit" et "Mes audits"

    •  Menu "Actions" : renommer le bouton" Télécharger l'audit" en "Télécharger la grille d'audit"
  • Pages "Mes audits"

    • [ ] Revoir l'espacement entre les colonne "Type d'audit" et "Créé le"
    • [ ] Alignement le libellé "Type d'audit" avec sa colonne
  • Modale "Partager l'audit"

    •  Ajouter un "." à la fin la mention "Vous êtes la seule personne à pouvoir accéder à l’audit et le modifier"
    •  Ajouter un "." à la fin la mention "Toute personne disposant du lien peut accéder à l’audit et le modifier"
    • Renommer le feedback "Lien de partage copié" en "Lien copié"
    • Utiliser une icône "Link-line" pour le bouton "Copier le lien de partage"
    • Étendre la modale uniquement vers le bas (plutôt que vers le haut et le bas) lors du passage de l'audit en mode "public" (et inversement lors du passage en mode "privé"). Cf proto.
    • [ ] Réduire l'espacement entre le bouton "Copier le lien de partage" et le bas de la modale à 32 px.
  • Page « Paramètres de l’audit »

    • Ne plus afficher la section « Informations personnelles » lorsqu'un audit est associé à un compte
      • Empêcher l’accès à cette section en navigation privée
      • Appliquer cette règle que l’audit soit public ou privé
  • Page "Accès restreint"

    •  Réduire la dernière phrase du paragraphe à : "Pour accéder à l’audit, contactez le propriétaire."

@hissalht hissalht temporarily deployed to ara-1404-rendre-les-lie-nca2kw March 25, 2026 14:58 Inactive
@emma11y emma11y marked this pull request as draft March 25, 2026 15:48
@bellangerq bellangerq force-pushed the 1404-rendre-les-liens-des-audits-privés-mode-connecté branch from aca675a to ee07d08 Compare May 20, 2026 15:54
@bellangerq bellangerq marked this pull request as ready for review May 22, 2026 13:37
@bellangerq

bellangerq commented Jun 3, 2026

Copy link
Copy Markdown
Collaborator

Retours au 3 juin 2026

Regroupe l'ensemble des retours sur la fonctionnalité :

  • Pages "Mes audits"

    •  Revoir l'espacement entre les colonne "Type d'audit" et "Créé le"
    •  Alignement le libellé "Type d'audit" avec sa colonne
  • Modale "Partager l'audit"

    • Étendre la modale uniquement vers le bas (plutôt que vers le haut et le bas) lors du passage de l'audit en mode "public" (et inversement lors du passage en mode "privé"). Cf proto. Cacher visuellement et accessiblement le contenu.
    • Réduire l'espacement entre le bouton "Copier le lien de partage" et le bas de la modale à 32 px.
  • Menu "Actions" : toute personne disposant du lien d'un audit non attaché à un compte -> adresse e-mail de l'audit rattachée à aucun compte

    • Désactiver le bouton "Dupliquer"
    • Afficher une mention sous le bouton désactivé : "Disponible uniquement avec un compte".
    • Vérifier le cas pour le bouton "Partager"
image
  • Débug
    • Corriger le point d'api de debug : un audit créé est inaccessible par défaut ("Accès restreint").

@bellangerq bellangerq force-pushed the 1404-rendre-les-liens-des-audits-privés-mode-connecté branch from e092665 to a9013b9 Compare June 4, 2026 15:39
@AdrienMuzyczka

AdrienMuzyczka commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

Retour suite à test du 08/06 :

  •  Audit sans compte : ne jamais désactiver l'option de suppression dans le menu déroulant "Actions".
    Capture d’écran 2026-06-09 à 11 03 23

  •  Menu déroulant "Actions" page "Audit" : réduire la largeur du panneau déroulant à la largeur du libellé de l'option la plus large.

  •  Modale de partage : préciser dans le texte d'information affiché si l'audit est "Public" : "La modification d'un champ dans un audit par [...]"

@hissalht hissalht left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Il ne devrait pas y avoir de vérification côté front !

Si l’utilisateur actuel n’a pas accès à l’audit, l’API devrait renvoyer une erreur 401 (utilisateur pas connecté) ou 403 (utilisateur pas le propriétaire d’un audit privé).

Et ce sur toute les routes API d’un audit (mise à jour, suppression, etc)

Sinon les audits ne sont que invisibles sur l’appli mais restent pleinement accessibles via l’API.

J’aime bien la métaphore du panneau pour illustrer ça
Personnage de cartoon dont le chemin est bloqué par un panneau indiquand 'Keep out, or enter. im a sign not a cop'

EDIT: pour gérer ça côté back, plutôt que de faire la vérif dans chaque méthode du controlleur qui accède à un audit potentiellement privé. Y’a sûrement moyen de modifier le décorateur @AuditId() pour vérifier l’existence de l’audit ET si l’utilisateur actuel a accès à l’audit. Je sais pas trop comment exactement là, mais je pourrais regarder si vous voulez @bellangerq @emma11y

Comment thread confiture-rest-api/src/audits/audit.service.ts Outdated
Comment thread confiture-rest-api/src/audits/audits.controller.ts
Comment thread confiture-rest-api/src/audits/audits.controller.ts Outdated
Comment thread confiture-web-app/src/styles/dsfr-custom-icons/file-copy-2-line.svg Outdated
Comment thread confiture-web-app/src/composables/useCanAccessToAudit.ts Outdated
Comment thread confiture-web-app/src/components/layout/AccountHeader.vue Outdated
Comment thread confiture-web-app/src/components/audit/AuditSettingsForm.vue Outdated
Comment thread confiture-web-app/src/components/audit/ShareModal.vue
Comment thread confiture-web-app/src/components/ui/CopyBlock.vue Outdated
Comment thread confiture-web-app/src/composables/useCanAccessToAudit.ts Outdated
@bellangerq bellangerq force-pushed the 1404-rendre-les-liens-des-audits-privés-mode-connecté branch 3 times, most recently from d4692e2 to 64bc3e6 Compare June 18, 2026 09:20
emma11y and others added 29 commits June 24, 2026 09:53
@bellangerq bellangerq force-pushed the 1404-rendre-les-liens-des-audits-privés-mode-connecté branch from 6b2353a to ba0ecb4 Compare June 24, 2026 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rendre les liens des audits privés (mode connecté)

5 participants