Skip to content
This repository was archived by the owner on Apr 9, 2026. It is now read-only.

module memoire - #281

Merged
laem merged 47 commits into
masterfrom
module-memoire
Dec 16, 2024
Merged

laem merged 47 commits into
masterfrom
module-memoire

Conversation

@laem

@laem laem commented Nov 20, 2024

Copy link
Copy Markdown
Contributor

Description initiale

Pas encore fonctionnel : il ne faut garder que les paramètres de
simulation validés par l'utilisateur : _ vs *. La simulation est stockée via une URL de simulation qu'on peut sérialiser / désérialiser facilement, que ce soit dans le module ampleur ou dans la simulation générale. Ainsi un utilisateur qui aurait déjà fait son test MAR et viendrait utiliser le module ampleur aurait des infos déjà remplies.

Ensuite, il faut que l'on affiche un résumé de la situation mémorisée et
la possibilité de revenir en arrière. Je pensais adapter le "texte à trou" actuel et ajouter un bouton discret "Refaire une simulation" à la fin.

Vérifier également en pratique via codesandbox que l'iframe nous permet de faire ça, sans besoin d'un paramètre allowLocalStorage ou assimilé.

Enfin, on veut gérer l'injection de données depuis le site hôte pour que la simulation après CTA soit la plus courte possible. Est-ce que ça implique un élément d'UI pour expliquer ça ? Ou plutôt le gérer dans le module via les paramètres injectés qui décrivent l'annonce très brièvement ?

MAJ : du coup ça me semble pertinent d'intégrer dans cette PR la fonctionnalité des seuils de revenu plutôt que la saisie revenu. Ainsi on se met en cohérence avec le simulateur principal, on améliore l'UX (pas de crainte de divulguer trop d'infos et facilité à trouver le seuil plutôt que le revenu à l'€ près), chose qu'on devra de toutes façon toucher pour cette fonctionnalité de mémoire.

Avancement

  •  mettre au propre les composants
  •  conditionnalité de la question revenu qui devient seuils
  • marquer "touchés" les champs remplis
  • ne pas leur donner de valeur par défaut pour savoir qu'ils ont été touchés
  • stocker la situation
  • passer la situation à la suite
  • bug DPE qui change pas, et griser les DPE pas sélectionnables
  • Réflexion : mettre le DPE comme une question comme les autres ? Aussi, Martin trouve que choisir le DPE pour l'utilisateur n'est pas une bonne idée, par rapport au fait de montrer des scénarios
  • v1 de l'écran d'information sur le fait qu'on a lancé la simulation avec ses données
  • vérifier que la mémoire marche bien d'annonce en annonce sans bugs divers
  • bug qui affectait la v2 : l'année de construction n'est pas intégrée dans la simulation

Je ne sais pas si on peut gérer ça dans publicodes, avec des "non défini" par exemple. J'ai testé sans succès. Sinon on peut le faire en JS dans le module, même si c'est moins propre.

  • bug : le OK fait sauter la question courante ? À vérifier

  • seuils de revenu à ne pas préremplir

  • icône de localisation dans le titre de l'annonce ?

  • pas de placeholder à mettre sur le nb de personnes, et +- plus gros ?

  • mettre à jour le CHANGELOG

  • lien de préprod de branche

  • parler des nouvelles variables

Par la suite

Cacher les aides par avant les saisies et les remplacer par un truc visuel qui donne envie sans promettre des aides.

laem added 2 commits November 20, 2024 15:49
Pas encore fonctionnel : il ne faut garder que les paramètres de
simulation validés par l'utilisateur : _ vs *.

Ensuite, il faut que l'on affiche un résumé de la situation mémorisée et
la possibilité de revenir en arrière.
@vercel

vercel Bot commented Nov 20, 2024

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
reno ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 16, 2024 10:29am

@laem

laem commented Nov 25, 2024

Copy link
Copy Markdown
Contributor Author

Débats

Des débats ont eu lieu dans l'équipe sur deux points :

Faut-il afficher des résultats par défaut, ou attendre les réponses pour les afficher ?

Argument pour : attirer l'attention
Argument contre : les résultats sont des moyennes, donc faux pour la plupart des gens, avec le risque qu'ils n'interragissent pas avec le module parce qu'ils ont déjà eu l'impression d'avoir eu l'info.

[MT] De mon côté je pense qu'il faut choisir entre : aucun résultat, ou un bloc qui attire l'attention mais sans donner de résultats. Je trouve que le risque d'une lecture ultra rapide qui considérerait que c'est bon c'est simulé est trop grand, et nous donne des statistiques problématiques.

Faut-il rendre la question seuil dynamique ?

Argument pour : ne pas montrer de prime abord des seuils qui changeront, ce qui pourrait perturber l'utilisateur ("j'ai essayé de changer mon revenu mais ça marchait pas").
Argument contre : l'apparition dynamique pourrait faire peur d'être tombé sur une arnaque qui posera plus de questions encore.

[MT] Perso je ne pige pas trop l'argument contre avec le design proposé qui dit "dernière question".

@laem
laem marked this pull request as ready for review November 27, 2024 16:23
Comment thread app/module/AmpleurInputs.tsx

@morganmerzouk morganmerzouk left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Bravo 👍

Comment thread app/module/AmpleurQuestions.tsx Outdated
depuisModule,
setSearchParams,
situation,
remaining: nextQuestions.length,

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Sur le module (en passant par notre page /module):

  • j'ai rempli tout sauf le revenu fiscal
  • j'ai cliqué "Découvrir mes aides"
  • j'ai eu le message "Il reste 6 questions à remplir" et la question "ménage . commune"
  • en répondant je suis arrivé directement sur la page d'éligibilité

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Je crois que c'est réglé avec l'apparition du CTA seulement quand le revenu est renseigné ?

@laem
laem merged commit bbaf7df into master Dec 16, 2024
@laem
laem deleted the module-memoire branch December 16, 2024 15:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants