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

[Dépôt de besoin] Ajouter une stat du nombre de structures qui ont cliqué OU vu #927

Conversation

raphodn
Copy link
Contributor

@raphodn raphodn commented Sep 28, 2023

Quoi ?

Sur le modèle Tender, il y a une property siae_detail_display_date_count_all.
Mais il y avait une incohérence dans l'affichage du détail du besoin.

Modifications apportées :

  • renommé la property en siae_email_link_click_date_or_detail_display_date_count
  • rajouté la stat siae_email_link_click_or_detail_display_count dans le calcul de la queryset with_siae_stats()

@raphodn raphodn self-assigned this Sep 28, 2023
@raphodn raphodn changed the title [Dépôt de besoin] Ajouter un stat du nombre de structures qui ont cliqué OU vues [Dépôt de besoin] Ajouter un stat du nombre de structures qui ont cliqué OU vu Sep 28, 2023
@raphodn raphodn force-pushed the raphodn/tender-stat-siae-email-link-click-or-detail-display-count branch from 6794476 to 04d5a6c Compare September 28, 2023 10:21
@raphodn raphodn changed the title [Dépôt de besoin] Ajouter un stat du nombre de structures qui ont cliqué OU vu [Dépôt de besoin] Ajouter une stat du nombre de structures qui ont cliqué OU vu Sep 28, 2023
@raphodn raphodn requested a review from madjid-asa September 28, 2023 10:29
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.

LGTM

@raphodn raphodn merged commit e80350d into master Oct 2, 2023
4 checks passed
@raphodn raphodn deleted the raphodn/tender-stat-siae-email-link-click-or-detail-display-count branch October 2, 2023 12:36
raphodn added a commit that referenced this pull request Jan 8, 2024
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.

2 participants