search

CAUCA-9-1-1 / survip

SURVI-Prevention Desktop
2 stars 3 forks source link

Questionnaire - For groups of questions the minimum and maximum occurrence does not work #549

Closed jocelyncardinal closed 5 years ago

jocelyncardinal commented 5 years ago

Describe the bug For groups of questions the minimum and maximum occurrence does not work

To Reproduce Set minimum and maximum occurrence mix max to zero ... Capture d’écran, le 2019-05-23 à 11.59.08.png

... in app mobile it is impossible to make zero occurence ...

Capture d’écran, le 2019-05-23 à 12.19.32.png

Expected behavior Should use properly the minimum and maximum occurrence that has been set. Note that If maximum is set to zero, it would mean "no maximum limit".

Work around For the time being as work around we added an option "No item" that the user can to select ... Capture d’écran, le 2019-05-23 à 12.18.42.png

jocelyncardinal commented 5 years ago

Idea: When the minimum occurrence is zero, we could initialy have no item. Then the user must click the "+" button to add one or click "next" to go to the next question if there is no item to be added.

iamkinetic commented 5 years ago

@philipperobertgh Dis-moi donc ce que tu penses de ça svp

philipperobertgh commented 5 years ago

SI les limites d'occurrences ne fonctionnent pas dans le mobile, il faut les corriger. Donc je suis d'accord que si l'occurrence minimale est de 0 on devrait pouvoir passer à la question suivante sans ajouter d'item.

jocelyncardinal commented 5 years ago

Si l'occurrence minimale est de 0 on devrait pouvoir passer à la question suivante sans ajouter d'item mais il faudrait aussi conserver la possibilité d’ajouter un item (avec le petit bouton «+ » en haut à droite)

iamkinetic commented 5 years ago

Ok, on va améliorer ça aussi. Les changements se feront uniquement dans l'application mobile, donc ce sera plus long avant que tu aies cette correction parce que je suis en train de faire la migration vers Ionic 4 et que ça me prendra encore au moins une semaine. Lorsque ce sera terminé, on pourra faire la correction.

iamkinetic commented 5 years ago

Blocked by survip-mobile#295.

iamkinetic commented 5 years ago

Je ne crois pas que ce changement sera fait avant le mois de juillet. Je dois d'abord terminer l'upgrade vers Ionic 4.

jocelyncardinal commented 5 years ago

Ce n’est pas une issue bloquante - Le work-around est OK et on peut attendre sans problème pour juillet

iamkinetic commented 5 years ago

FYI je me suis remis à l'upgrade vers ionic 4 vendredi et je travaillerai là dessus cette semaine.

iamkinetic commented 5 years ago

PRE-413

philipperobertgh commented 5 years ago

Issue moved to CAUCA-9-1-1/survip-mobile #338 via ZenHub