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

feat: Ajout d'attributs de contact pour les acheteurs inscrits #1468

Merged
merged 6 commits into from
Nov 7, 2024

Conversation

chloend
Copy link
Contributor

@chloend chloend commented Oct 17, 2024

Quoi ?

Ajout de trois attributs lors de la création de contact dans la liste #10 (Acheteurs inscrits) de Brevo :

  • MONTANT_BESOIN_ACHETEUR
  • TYPE_BESOIN_ACHETEUR
  • TYPE_VERTICALE_ACHETEUR

Pourquoi ?

Pour segmenter les envois d'email et ne pas multiplier les listes sur Brevo.

Comment ?

En ajoutant un argument tender_id dans la méthode add_to_contact_list définie dans emails.py. Cette dernière est appelée dans les trois méthodes de création de Tender (done, perform_create et csrf_failure) afin de récupérer l'id du tender qui permettra d'associer les informations du tender à l'utilisateur de type "acheteur".

Captures d'écran (optionnel)

contact_attributs

Autre (optionnel)

Tests :

  • Il manque un test sur la méthode csrf_failure car il était compliqué de récupérer l'erreur csrf dans le test pour la traiter et permettre la création du tender.
  • J'ai ajouté un User de type BUYER dans setUpTestData donc j'ai légèrement modifié test_user_with_valid_api_key_can_create_tender pour que les assertEqual du nombre de User passent.

lemarche/utils/emails.py Outdated Show resolved Hide resolved
lemarche/utils/apis/api_brevo.py Outdated Show resolved Hide resolved
lemarche/utils/apis/api_brevo.py Outdated Show resolved Hide resolved
Copy link
Contributor

@madjid-asa madjid-asa left a comment

Choose a reason for hiding this comment

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

Juste une petite optimisation de requête à faire mais sinon c'est très bien, bravo :D

lemarche/utils/apis/api_brevo.py Outdated Show resolved Hide resolved
lemarche/utils/apis/api_brevo.py Outdated Show resolved Hide resolved
lemarche/utils/apis/api_brevo.py Outdated Show resolved Hide resolved
lemarche/www/tenders/views.py Outdated Show resolved Hide resolved
lemarche/utils/emails.py Outdated Show resolved Hide resolved
lemarche/utils/emails.py Outdated Show resolved Hide resolved
lemarche/utils/emails.py Outdated Show resolved Hide resolved
lemarche/utils/apis/api_brevo.py Outdated Show resolved Hide resolved
@chloend chloend merged commit e508900 into master Nov 7, 2024
8 checks passed
@chloend chloend deleted the chloend/contact-attributes branch November 7, 2024 14:23
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.

3 participants