Skip to content

Ajoute une relation dans le schéma entre Audit et User#1475

Open
hissalht wants to merge 4 commits intomainfrom
auditor-email-test3
Open

Ajoute une relation dans le schéma entre Audit et User#1475
hissalht wants to merge 4 commits intomainfrom
auditor-email-test3

Conversation

@hissalht
Copy link
Copy Markdown
Collaborator

closes #1463

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é).

@hissalht hissalht temporarily deployed to ara-auditor-email-test3-c9k4or April 17, 2026 23:58 Inactive
@hissalht hissalht added the DX Concerne l'expérience développeur label Apr 18, 2026
Comment thread confiture-rest-api/prisma/migrations/migration_lock.toml
Comment thread confiture-rest-api/src/audits/audit.service.ts Outdated
return pagesWithSlug;
}

findPopulatedAuditWithUniqueId(editUniqueId: string): Promise<AuditDto> {
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.

Du coup j'imagine que ça sera remplacé dans l'autre PR par la fonction unique pour chercher un audit.

Comment thread confiture-rest-api/src/audits/audit.service.ts Outdated
Comment thread confiture-rest-api/src/audits/audit.service.ts
...(originalAudit.auditorEmail && {
auditor: {
connect: {
// FIXME: shouldnt the new audit be under the user who duplicated the audit ?
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.

Oui je pense, à vérifier avec @benoitdequick et @AdrienMuzyczka peut-être.

Mais je dirais :

  • Si connecté : auditorEmail = email de l'utilisateur connecté.
  • Si pas connecté : auditorEmail = email de l'audit.

@hissalht hissalht temporarily deployed to ara-auditor-email-test3-c9k4or April 22, 2026 13:21 Inactive
@hissalht hissalht force-pushed the auditor-email-test3 branch from 96fee8a to b4b30c3 Compare April 30, 2026 07:12
@hissalht hissalht had a problem deploying to ara-auditor-email-test3-c9k4or April 30, 2026 07:12 Failure
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DX Concerne l'expérience développeur

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Revoir l'association des modèles de la BDD Audit <> User

2 participants