Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEATURE] Ajout de l'API getLegalDocumentStatusByUserId dans legal-document context (PIX-15581) #10786

Merged

Conversation

bpetetot
Copy link
Contributor

@bpetetot bpetetot commented Dec 11, 2024

🎄 Problème

Dans le cadre de la nouvelle gestion du versioning des CGUs pour Pix Orga, nous avons besoin d’une API permettant de récupérer l'état d’acceptation d’un document légal pour un utilisateur.

🎁 Proposition

Ajouter une API getLegalDocumentStatusByUserId permettant récupérer le status d'un document légal pour un utilisateur

  • Si le feature toggle newLegalDocumentsVersioning est activé: calculer le status à partir du nouveau modèle de gestion de documents légaux.
  • Si le feature toggle newLegalDocumentsVersioning est désactivé: calculer le status à partir des attributs Pix Orga CGU de la table users.

Retour de l'API avec un modèle LegalDocumentStatus:

  • status: accepted, requested ou update-requested
  • acceptedAt: date d’acceptation (uniquement si accepted)
  • documentPath: chemin du document (pour pointer vers la page pix-site du document)

🧦 Remarques

  • Ajout d'une factory databaseBuilder pour legal-document-version-user-acceptances
  • Ajout de la vérification des attributs type et service dans le use case accept-legal-document-by-user-id
  • Changement d'ordre des propriétés des arguments pour que le code des legal documents soit plus logique à utiliser

🎅 Pour tester

  • L'API n'est pas encore utilisée, s'appuyer sur les tests d'intégration et unitaire.
  • Un série de test fonctionnels sera réalisée lors de son intégration.

@bpetetot bpetetot self-assigned this Dec 11, 2024
@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@bpetetot bpetetot force-pushed the pix-15581-add-api-to-get-legal-document-status branch 7 times, most recently from 0784c49 to 5a8ed3c Compare December 18, 2024 10:29
@bpetetot bpetetot marked this pull request as ready for review December 18, 2024 10:40
@bpetetot bpetetot force-pushed the pix-15581-add-api-to-get-legal-document-status branch from 5a8ed3c to 0791e81 Compare December 19, 2024 14:25
Copy link
Contributor

@lego-technix lego-technix left a comment

Choose a reason for hiding this comment

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

✅ Lu et validé avec succès

@lego-technix lego-technix force-pushed the pix-15581-add-api-to-get-legal-document-status branch 2 times, most recently from ebcd76a to b64d4f6 Compare December 20, 2024 15:14
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-15581-add-api-to-get-legal-document-status branch from b64d4f6 to 10c267c Compare December 20, 2024 15:24
@pix-service-auto-merge pix-service-auto-merge merged commit fa5773b into dev Dec 20, 2024
8 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-15581-add-api-to-get-legal-document-status branch December 20, 2024 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants