MTES-MCT / Docurba

4 stars 4 forks source link

Paramétrage fenêtres NPS #552

Open CeliaVermicelli opened 2 months ago

CeliaVermicelli commented 2 months ago

@Octaviedebs

NPS FRP

Conditions d'affichage :

NPS PAC

conditions d'affichage :

Condition d'affichage par user :

Notre question en suspens :

On voudrait créer 2 uniques formulaires mais pour autant avoir des infos de la personne ayant donné le feedback ;

Donc est-ce que c'est possible de faire remonter :

ça ferait une intégration Tally>Gsheet

(car il me semble que ça n'est pas possible de faire une intégration Tally<Docurba)

Reste à faire ensemble jeudi :

Octaviedebs commented 2 months ago

@CeliaVermicelli concernant Condition d'affichage par user : limiter à affichage de la fenêtre 1 fois toutes les 3 semaines (limiter la saturation des user qui sont sur le PAC ET sur une FRP) - je ne suis pas certaine que ce soit utile de limiter autant :

En revanche, en relisant je me dis que ce serait plus simple au lieu de = apparait à chaque navigation jusqu'à 3 fois max si non rempli j'aurais plutôt mis juste = apparait jusqu'à 3 fois max si non rempli ( en gros si tu fermes la fenêtre du NPS, ça disparait, mais si tu retournes sur la page là la pop up apparait à nouveau => et ce maximum 3 fois)

Tu en penses quoi ?

A voir également ce qui est le + simple/logique comme conditions à implémenter côté tech

CeliaVermicelli commented 2 months ago

limiter à affichage de la fenêtre 1 fois toutes les 3 semaines (limiter la saturation des user qui sont sur le PAC ET sur une FRP) - je ne suis pas certaine que ce soit utile de limiter autant

Je trouvais utile pour ne pas les saturer car quand on avait mis en place cette fenetre j'avais tout de suite eu des remarques des ddt "c'est embêtant cette fenetre qu'il faut fermer plusieurs fois" :/

j'aurais plutôt mis juste = apparait jusqu'à 3 fois max si non rempli ( en gros si tu fermes la fenêtre du NPS, ça disparait, mais si tu retournes sur la page là la pop up apparait à nouveau => et ce maximum 3 fois)

ah c'est comme ça que je l'avais compris donc alignée :)

UngererFabien commented 1 month ago

Hello ! Je faisais des tests et j'ai remarqué quelques points

  1. Le trigger automatique de Tally n'a pas l'air de fonctionner. Donc je l'ai déclancher à la main. Par contre il faut test si le param doNotShowAfterSubmit fonctionne encore.
  2. Les deux forms ont l'air dêtre les même ?
  3. On peu ajouter l'email dans un hidden field du formulaire mais l'intégration sera super lourdingue. Est ce que c'est vraiment important ? https://tally.so/help/hidden-fields
CeliaVermicelli commented 1 month ago

1- ça marche, on testera ça 2 et 3- oui les form sont les mêmes. Idéalement :

Est-ce que tu vois d'autres manières d'obtenir ces infos sans que ça soit trop lourd d'intégration ?

Octaviedebs commented 1 month ago

Alors, j'ai testé en pré-prod et sauf erreur de ma part mais j'attends les retours de @CeliaVermicelli avant. dela passer en KO =

CeliaVermicelli commented 1 month ago

J'aurais bien voulu vérifier ce qui se passe quand on répond effectivement au form. @Octaviedebs je passe en KO du coup

Capture d’écran 2024-06-04 à 11 34 40 AM

Octaviedebs commented 3 weeks ago

@UngererFabien Suite à ton slack, et cf la relecture des commentaire sur cette carte :

Ce commentaire : Image

Auquel @CeliaVermicelli a répondu : Image

Du coup qu'est ce qui n'était pas clair sur ce point ? Les weekly dev/ops servent justement à s'aligner à 100% donc hésite pas quand on est pas assez clair et/ou si tu préfères qu'on le reformule plus clairement ou autres retours

Octaviedebs commented 3 weeks ago

Résumé post call Fabien = pas de délay

Donc juste maximum de 3 fois par form (à chaque reload)

Comme on s'était dit !

Octaviedebs commented 3 weeks ago

NPS FRP

https://github.com/MTES-MCT/Docurba/assets/144041896/3112d813-5f87-4469-93c9-faf35456826a

Octaviedebs commented 3 weeks ago

NPS PAC

J'ai réussi à tester sur un PAC que j'ai créé :

En revanche il y a un doublon ❌ :

CE JAM : https://jam.dev/c/4a9240a6-8bb2-42bb-9486-6a0e5a5e5014

Par ailleurs, la pop up n'apparait bien plus si rempli ✅

CeliaVermicelli commented 3 weeks ago

✅ NPS FRP :

❌ NPS PAC :

Octaviedebs commented 2 weeks ago

@UngererFabien Comment tester pour les BE/CT en pré prod ? du coup on vérifiera que le NPS FRP pop bien que quand ce sera en prod je pense

Octaviedebs commented 2 weeks ago

TOUT OK pour NPS PAC & NPS FRP Plus de doublons / ça apparait bien 3 fois que si non rempli (sinon n'apparait plus)

Mise en prod = tester pour NPS FRP côté side collectivité