search

betagouv / mission-transition-ecologique

Application Transition Ecologique des Entreprises
https://mission-transition-ecologique.beta.gouv.fr/
GNU Affero General Public License v3.0
7 stars 7 forks source link

Improvement: GetDailyOppportunityByContactId #994

Open yvalentin opened 3 months ago

yvalentin commented 3 months ago

La méthode du backend GetDailyOppportunityByContactId semble pouvoir être amélioré afin d'éviter de récupérer tous les deals et de les filtrer via un for sur un contact.

L'api brevo v3/crm/deals permet de passer en filtre le contactId permettant d'avoir en retour uniquement des deals du contactId

Voir l'appel d'api ici

libs/backend-ddd/src/opportunity/infrastructure/api/brevo/brevoAPI.ts:67

En profiter pour vérifier l'utilité du param endDate

ttdm commented 2 months ago

C'est effectivement une amélioration de perfs. Après, avec le filtrage sur la date, on n'a jamais plus de 10éléments dans la réponse de l'api brevo donc c'est minime. ça pourrait devenir plus important si on passe à >1000 demandes journalières.