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

Fix answered question bug (NGC-913) #589

Merged
merged 7 commits into from
Jun 18, 2024
Merged

Conversation

Clemog
Copy link
Contributor

@Clemog Clemog commented Jun 4, 2024

No description provided.

Copy link

vercel bot commented Jun 4, 2024

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

Name Status Preview Updated (UTC)
nosgestesclimat ✅ Ready (Inspect) Visit Preview Jun 18, 2024 1:22pm

@Clemog Clemog changed the title WIP: fix answered question bug WIP: fix answered question bug (NGC-913) Jun 4, 2024
Copy link

Copy link

github-actions bot commented Jun 4, 2024

Report for the pull request #589


🌐 Translation status

UI's texts

Language Nb. missing translations Status
en
Missing 2 UI texts ⬇️
    Plus de 40 acteurs relaient <2>ou</2> ont relayé Nos Gestes Climat à travers <4>l’intégration du calculateur</4> sur leur site internet <8>ou</8> sa diffusion via <11>des campagnes</11> (mail, réseaux sociaux et/ou affichage). C’est majoritairement grâce à eux que nous sensibilisons près de 2 000 nouvelles personnes en moyenne chaque jour et nous les en remercions.,Plus de 40 acteurs relaient <2>ou</2> ont relayé Nos Gestes Climat à travers <4>l’intégration du calculateur</4> sur leur site internet <8>ou</8> sa diffusion via <11>des campagnes</11> (mail, réseaux sociaux et/ou affichage). C’est majoritairement grâce à eux que nous sensibilisons près de 2 000 nouvelles personnes en moyenne chaque jour et nous les en remercions. Un grand-père et sa petite-fille au cinéma, mangeant du pop-corn.,Un grand-père et sa petite-fille au cinéma, mangeant du pop-corn.
es
Missing 2 UI texts ⬇️
    Plus de 40 acteurs relaient <2>ou</2> ont relayé Nos Gestes Climat à travers <4>l’intégration du calculateur</4> sur leur site internet <8>ou</8> sa diffusion via <11>des campagnes</11> (mail, réseaux sociaux et/ou affichage). C’est majoritairement grâce à eux que nous sensibilisons près de 2 000 nouvelles personnes en moyenne chaque jour et nous les en remercions.,Plus de 40 acteurs relaient <2>ou</2> ont relayé Nos Gestes Climat à travers <4>l’intégration du calculateur</4> sur leur site internet <8>ou</8> sa diffusion via <11>des campagnes</11> (mail, réseaux sociaux et/ou affichage). C’est majoritairement grâce à eux que nous sensibilisons près de 2 000 nouvelles personnes en moyenne chaque jour et nous les en remercions. Un grand-père et sa petite-fille au cinéma, mangeant du pop-corn.,Un grand-père et sa petite-fille au cinéma, mangeant du pop-corn.

FAQ's questions

Language Nb. missing translations Status
en Ø ✔️
es Ø ✔️

You will find more information about the translation in the dedicated file.

@Clemog Clemog changed the title WIP: fix answered question bug (NGC-913) Fix answered question bug (NGC-913) Jun 5, 2024
@@ -183,7 +142,6 @@ export default function useQuestions({

const isInMissingVariables =
Object.keys(rawMissingVariables).includes(foldedStep)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Plus ça va et plus je me dis que cette idée que j'ai eue à l'époque d'utiliser les rawMissingVariables n'est peut-être pas la meilleure..

On a un cas "non couvert" qui peut arriver de temps en temps dans le cas ou certaines questions conditionnées sont des les rawMissingVariables:

Ex:

La question des efforts pour la préservation des appareils est applicable si le nombre d'appareils est supérieur à 0. Par défaut c'est le cas donc c'est dans les rawMissingVariables. Or, si j'ai fini le test et que finalement, je me rends compte que j'ai que des appareils de plus de 10 ans, je sélectionne "aucun" mais la question d'après reste celle sur la présevation et là PATATRA

Je me dis en écrivant qu'on devrait peut-être avoir des les rawMissingVariables seulement les missingVariables de bilan qui ne sont pas conditionnées

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Je viens de tester un truc, tu me diras ce que tu en penses

Copy link
Contributor

Choose a reason for hiding this comment

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

Soyons honnête, je ne suis pas sur de comprendre

Copy link
Contributor

Choose a reason for hiding this comment

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

Soyons encore plus honnête, je suis sur de ne pas comprendre

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Je vous montre un exemple lundi !

@bjlaa
Copy link
Contributor

bjlaa commented Jun 10, 2024

ça fonctionne bien de mon côté, il faudrait juste tester avec le localSotrage de Jeanne pour voir si ça résoud également le problème

@Clemog Clemog merged commit c88f732 into preprod Jun 18, 2024
6 checks passed
@Clemog Clemog deleted the fix-answered-question-bug branch June 18, 2024 13:23
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.

3 participants