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

Update de la fonction SARAH.askme() #3

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

tilleul
Copy link

@tilleul tilleul commented Apr 27, 2015

La doc (wiki) indique pour la fonction SARAH.askme():
"S'il n'y a aucune réponse après timeout x 2, ou 8 secondes, alors la fonction de callback est appelée avec false en paramètre"

Mais en réalité, il y avait une question avec timeout x 1 puis répétition de la question avec un timeout de 16 secondes.

La modif de code permet:

  • si timeout est positif, que Sarah pose la question deux fois avec le timeout indiqué
  • si timeout est négatif, que Sarah pose la question UNE fois avec le timeout indiqué
  • si timeout vaut zéro, Sarah pose la question UNE fois avec un timeout par défaut de 8 secondes

A part le délai "par défaut" qui revient à 8 secondes comme décrit dans la doc wiki, ce changement ne devrait avoir aucun impact sur les différents plugins déjà publiés.

La doc (wiki) indique pour la fonction SARAH.askme():
"S'il n'y a aucune réponse après timeout x 2, ou 8 secondes, alors la fonction de callback est appelée avec false en paramètre"

Mais en réalité, il y avait une question avec timeout x 1 puis répétition de la question avec un timeout de 16 secondes.

La modif de code permet:
- si timeout est positif, que Sarah pose la question deux fois avec le timeout indiqué
- si timeout est négatif, que Sarah pose la question UNE fois avec le timeout indiqué
- si timeout vaut zéro, Sarah pose la question UNE fois avec un timeout par défaut de 8 secondes
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.

1 participant