search

multiplie-fr / minimit-beta

Dev and Beta version of the MiniMit software
10 stars 2 forks source link

Correction sur Eliza #34

Closed Pascalolila closed 1 year ago

Pascalolila commented 1 year ago

Quand on corrige, seule la portion saisie APRÈS la correction est envoyé à Eliza

Pascalolila commented 1 year ago

Si je dis Life sucks ça me répond YOU SAY #‾# ?

Pascalolila commented 1 year ago

Oui

Le 29 mai 2023 à 19:05, multiplié @.***> a écrit :

Est ce que CON/FIN fonctionne ?

— Reply to this email directly, view it on GitHub https://github.com/multiplie-fr/minimit-beta/issues/34#issuecomment-1567354247, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATUNJILSXRTPAXPHA2BR5ZDXITJN5ANCNFSM6AAAAAAYS7KM5A. You are receiving this because you authored the thread.

multiplie-fr commented 1 year ago

Bug YOU SAY -> Fixed

multiplie-fr commented 1 year ago

J'ai l'impression que la Correction fout quand même pas mal le bordel. On remplace par Annulation ?

choueche commented 1 year ago

c'est depuis la nouvelle architecture

la function wait_for_user_action()

remet userInput à vide alors que dans la function correctionEliza j'essaie manipuler userInput

je ne vois pas comment résoudre ca.

Le lun. 29 mai 2023 à 19:45, multiplié @.***> a écrit :

J'ai l'impression que la Correction fout quand même pas mal le bordel. On remplace par Annulation ?

— Reply to this email directly, view it on GitHub https://github.com/multiplie-fr/minimit-beta/issues/34#issuecomment-1567380640, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGHTT5GGSJRBV4L2TM6FI73XITOFFANCNFSM6AAAAAAYS7KM5A . You are receiving this because you were assigned.Message ID: @.***>

choueche commented 1 year ago

j'avais pris exemple la dessus où il y a une variable "fin" qui remet userInput à "". c'est un peu naze de ne pas gérer la correction, mais un peu tard pour se lancer dans cette modif maintenant mais je pense c'est prioritaire pour la v1

Le lun. 29 mai 2023 à 19:55, Cecile Adam @.***> a écrit :

c'est depuis la nouvelle architecture

la function wait_for_user_action()

remet userInput à vide alors que dans la function correctionEliza j'essaie manipuler userInput

je ne vois pas comment résoudre ca.

Le lun. 29 mai 2023 à 19:45, multiplié @.***> a écrit :

J'ai l'impression que la Correction fout quand même pas mal le bordel. On remplace par Annulation ?

— Reply to this email directly, view it on GitHub https://github.com/multiplie-fr/minimit-beta/issues/34#issuecomment-1567380640, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGHTT5GGSJRBV4L2TM6FI73XITOFFANCNFSM6AAAAAAYS7KM5A . You are receiving this because you were assigned.Message ID: @.***>

multiplie-fr commented 1 year ago

Hello,

Donc je supprime les correction partout ? Ou tu le fais ?

Le lun. 29 mai 2023, 20:02, choueche @.***> a écrit :

j'avais pris exemple la dessus où il y a une variable "fin" qui remet userInput à "". c'est un peu naze de ne pas gérer la correction, mais un peu tard pour se lancer dans cette modif maintenant mais je pense c'est prioritaire pour la v1

Le lun. 29 mai 2023 à 19:55, Cecile Adam @.***> a écrit :

c'est depuis la nouvelle architecture

la function wait_for_user_action()

remet userInput à vide alors que dans la function correctionEliza j'essaie manipuler userInput

je ne vois pas comment résoudre ca.

Le lun. 29 mai 2023 à 19:45, multiplié @.***> a écrit :

J'ai l'impression que la Correction fout quand même pas mal le bordel. On remplace par Annulation ?

— Reply to this email directly, view it on GitHub < https://github.com/multiplie-fr/minimit-beta/issues/34#issuecomment-1567380640 , or unsubscribe < https://github.com/notifications/unsubscribe-auth/AGHTT5GGSJRBV4L2TM6FI73XITOFFANCNFSM6AAAAAAYS7KM5A

. You are receiving this because you were assigned.Message ID: @.***>

— Reply to this email directly, view it on GitHub https://github.com/multiplie-fr/minimit-beta/issues/34#issuecomment-1567392220, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUEHLVZ7PEGDMY4QUUOR4IDXITQDHANCNFSM6AAAAAAYS7KM5A . You are receiving this because you were assigned.Message ID: @.***>

choueche commented 1 year ago

WAIT 10 minutes j'ai chopé le bug

Le mar. 30 mai 2023 à 12:56, multiplié @.***> a écrit :

Hello,

Donc je supprime les correction partout ? Ou tu le fais ?

Le lun. 29 mai 2023, 20:02, choueche @.***> a écrit :

j'avais pris exemple la dessus où il y a une variable "fin" qui remet userInput à "". c'est un peu naze de ne pas gérer la correction, mais un peu tard pour se lancer dans cette modif maintenant mais je pense c'est prioritaire pour la v1

Le lun. 29 mai 2023 à 19:55, Cecile Adam @.***> a écrit :

c'est depuis la nouvelle architecture

la function wait_for_user_action()

remet userInput à vide alors que dans la function correctionEliza j'essaie manipuler userInput

je ne vois pas comment résoudre ca.

Le lun. 29 mai 2023 à 19:45, multiplié @.***> a écrit :

J'ai l'impression que la Correction fout quand même pas mal le bordel. On remplace par Annulation ?

— Reply to this email directly, view it on GitHub < https://github.com/multiplie-fr/minimit-beta/issues/34#issuecomment-1567380640 , or unsubscribe < https://github.com/notifications/unsubscribe-auth/AGHTT5GGSJRBV4L2TM6FI73XITOFFANCNFSM6AAAAAAYS7KM5A

. You are receiving this because you were assigned.Message ID: @.***>

— Reply to this email directly, view it on GitHub https://github.com/multiplie-fr/minimit-beta/issues/34#issuecomment-1567392220, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUEHLVZ7PEGDMY4QUUOR4IDXITQDHANCNFSM6AAAAAAYS7KM5A . You are receiving this because you were assigned.Message ID: @.***>

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were assigned.Message ID: @.***>

Pascalolila commented 1 year ago

Ce bug était général, et génialement corrigé