Home : améliorer le wording de la section "Mon bénévolat" #728
Conversation
| $remaining = $this->due_duration_by_cycle - $member->getTimeCount($cycle_end); | ||
|
|
||
| // also take into account planned shifts | ||
| if ($this->use_card_reader_to_validate_shifts) { |
There was a problem hiding this comment.
Pourquoi mettre ce if ici ?
J'aurais plutot vu la condition :
true si le compteur est négatif OU pas 3h de créneaux programmés sur le cycle courant
There was a problem hiding this comment.
Ca permettrait d'être indépendant du code de validation des créneaux.
There was a problem hiding this comment.
Niveau implémentation, il faudrait regarder les futurs créneaux non validés.
(partant du principe que dans le cas not use_card_reader_to_validate_shifts, les créneaux sont validés par défaut)
There was a problem hiding this comment.
je peux enlever le if, mais c'était pour éviter de faire des requêtes DB pour les autres coop qui n'utilisent pas ce système, et donc qui ne peuvent pas avoir de créneau à venir qui n'apparaisse pas encore dans le compteur temps.
Si je comprends bien, tu proposes d'enlever l'appel à getTimeCount, et regarder plutôt la durée total des créneaux du cycle en cours ?
| <p> | ||
| {% if beneficiariesWhoCanBook | length > 1 %} | ||
| {% for b in beneficiariesWhoCanBook %} {{ b.firstname }} {% if loop.index < loop.length %} et {% endif %}{% endfor %}peuvent | ||
| {% else %} | ||
| {{ (beneficiariesWhoCanBook | first).firstname }} peut | ||
| {% endif %} encore | ||
| effectuer {{ duration_to_book | duration_from_minutes }}. | ||
| effectuer {{ cycleRemainingToBookExtra | duration_from_minutes }}. | ||
| </p> | ||
| {% elseif (allow_extra_shifts) %} |
There was a problem hiding this comment.
Remonter ce if d'un niveau. Si tu as fait plus de créneau alors cycleRemainingToBookExtra < 0. On ne rentre donc jamais dans ce if.
There was a problem hiding this comment.
hmm je suis un perdu là pour être honnête
raphodn
force-pushed
the
raphodn/home-dashboard-shift-cleanup
branch
from
c050bb3 to
814de16
Compare
|
je vais fermer cette PR et remplacer par plusieurs petites PR séparées |
Quoi ?
Pour les épiceries qui ont
use_card_reader_to_validate_shifts=trueActuellement la section "Mon bénévolat" ne prend pas en compte les créneaux réservés à l'avance.
Modifié
ShiftService.remainingToBook(member)pour prendre en compte les créneaux à venir du cycle.J'ai aussi remonté le message "exemption" pour qu'il apparaisse avant la section "Mon bénévolat"
Pourquoi ?
Clarifier les messages, en particulier pour les membres "fixe" qui ont déjà leur créneaux de générés à chaque début de cycle