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(forum): collecter les notations des fiches pratiques #677

Merged

Conversation

vincentporte
Copy link
Contributor

@vincentporte vincentporte commented Jun 13, 2024

Description

🎸 AJouter un formulaire de notation 5 étoiles dans les fiches pratiques,
🎸 Enregistrer les résultats par une vue HTMX
🎸 Notation anonyme possible
🎸 Une notation par forum et par session possible, si une notation est trouvée, l'afficher à la place du formulaire

🦺 en remplacement du formulaire TALLY, car la collecte de ces notations necessiterait la mise en place d'un appel webhook signé vers N8N, la verification de la signature dans N8N, l'implementation de DRF dans lacommunaute (+gestion de l'authentification), en plus de l'implémentation du modèle.

🦺 à venir dans des PR suivantes : 

  • afficher le nombre et la moyenne de notation de la fiche consultée
  • leaderboard des meilleures fiches
  • mise en avant de fiches par encore beaucoup notées

Type de changement

🎢 Nouvelle fonctionnalité (changement non cassant qui ajoute une fonctionnalité).
🎨 changement d'UI

Points d'attention

🦺 mise à jour de parse_response_to_soup pour gerer les url dans un attribut hx-post
🦺 adaptation pour tester la position des notes dans 3 endroits différents pour de l'AB testing

Captures d'écran

positionnement par défaut

image

AB Testing : 3 zones

image

image

@vincentporte vincentporte linked an issue Jun 13, 2024 that may be closed by this pull request
@vincentporte vincentporte added the python Pull requests that update Python code label Jun 13, 2024
@vincentporte vincentporte marked this pull request as ready for review June 13, 2024 15:57
lacommunaute/forum/models.py Show resolved Hide resolved
lacommunaute/templates/forum/partials/rating.html Outdated Show resolved Hide resolved
lacommunaute/templates/forum/partials/rating.html Outdated Show resolved Hide resolved
lacommunaute/forum/views.py Show resolved Hide resolved
lacommunaute/forum/models.py Show resolved Hide resolved
Copy link
Contributor

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

@vincentporte vincentporte removed the recette-jetable review_app label Jun 18, 2024
@vincentporte vincentporte merged commit 4c39290 into master Jun 18, 2024
12 checks passed
@vincentporte vincentporte deleted the 673-stats-collecter-la-notation-des-fiches-pratiques branch June 18, 2024 07:49
vincentporte pushed a commit that referenced this pull request Jul 4, 2024
🤖 I have created a release *beep* *boop*
---


##
[2.11.0](v2.10.0...v2.11.0)
(2024-07-03)


### Features

* **admin:** améliorer le filtrage des stats de forum
([#699](#699))
([cd0d301](cd0d301))
* **admin:** filtrage des stats de forum, ajout tri
([#701](#701))
([1da74df](1da74df))
* **forum_conversation:** filtrer les questions dans l'espace d'echanges
- stats de suivi
([#700](#700))
([bac6e55](bac6e55))
* **forum_conversation:** filtrer les questions dans l'espace d'échanges
([#681](#681))
([48cdd07](48cdd07))
* **forum:** Activation de l'AB Test Notation des forums
([#686](#686))
([c5699d4](c5699d4))
* **forum:** collecter les notations des fiches pratiques
([#677](#677))
([4c39290](4c39290))
* **home:** MVP zone editoriale sur la page d'accueil
([#703](#703))
([5e3f759](5e3f759))
* **home:** petites améliorations
([#689](#689))
([7471451](7471451))
* **notification:** nouveau système des notifications messages
([#697](#697))
([f2eedd4](f2eedd4))
* **stats:** collecter les stats d'activité hebdo des forums de la
documentation
([#691](#691))
([ab513a8](ab513a8))
* **stats:** page hebdo partenaires fiches pratiques
([#695](#695))
([f7f26b8](f7f26b8))


### Bug Fixes

* **cron.json:** erreur copier-coller sur send_notifications_daily.sh
([#705](#705))
([32f59ed](32f59ed))
* **forum:** ab test forum rating
([#687](#687))
([f0e9c2b](f0e9c2b))
* **forum:** ab test forum rating
([#688](#688))
([d3027e4](d3027e4))
* **notification:** mettre à jour le sent_at pour éviter les
notifications duplicates
([#707](#707))
([fca7f29](fca7f29))
* **stats:** derniere date collectée des stats matomo
([#684](#684))
([015e08b](015e08b))
* **stats:** gel du temps dans les tests de la vue
`StatistiquesPageView`
([#702](#702))
([df66524](df66524))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
python Pull requests that update Python code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[STATS] collecter la notation des fiches pratiques
3 participants