-
Notifications
You must be signed in to change notification settings - Fork 1
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
vincentporte
merged 18 commits into
master
from
673-stats-collecter-la-notation-des-fiches-pratiques
Jun 18, 2024
Merged
feat(forum): collecter les notations des fiches pratiques #677
vincentporte
merged 18 commits into
master
from
673-stats-collecter-la-notation-des-fiches-pratiques
Jun 18, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ase when rating already exists
vincentporte
commented
Jun 13, 2024
vincentporte
commented
Jun 14, 2024
🥁 La recette jetable est prête ! 👉 Je veux tester cette PR ! |
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
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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 :
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 attributhx-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
AB Testing : 3 zones