Modale au clic sur un créneau de la page planning #1122
Conversation
| </button> | ||
| {% endif %} | ||
| {{ form_end(shift_free_forms[shift.id]) }} | ||
| {% if shift_free_forms is defined %} |
There was a problem hiding this comment.
Je ne sais pas trop ce que sont les shift_free_forms, et donc je ne sais pas si vérifier leur présence est la meilleure façon de faire ici.
Idem pour les autres if … is defined plus bas :
shift_book_formsbucket_shift_add_formbucket_lock_unlock_form
There was a problem hiding this comment.
ce ne sont pas des free_form, mais des shift_free forms ;) Le créneaux libres sont une fonctionnalités offerte par Elefan que l'on n'utilise pas nous. Une variable existe pour activer la fonctionnalité.
| {% if is_granted("ROLE_ADMIN_PANEL") %} | ||
| <a href="{{ path('mail_bucketshift', { 'id': bucket.id }) }}" class="btn"> | ||
| <i class="material-icons left">mail</i>Envoyer un email | ||
| </a> | ||
| {% endif %} |
There was a problem hiding this comment.
Je ne suis pas sûr si le test sur ROLE_ADMIN_PANEL est le bon test pour savoir si on doit afficher le bouton Envoyer un email.
Je l'ai rajouté car un⋅e membre sans droits admin voit une page 403 si iel clique sur ce bouton.
There was a problem hiding this comment.
tu as probablement résolu un autre bug :). Ce rôle est peut-être pas suffisant par contre :)
| {% if is_granted("ROLE_SHIFT_MANAGER") %} | ||
| <a href="{{ path('bucket_edit', { 'id': bucket.id }) }}" class="btn deep-purple"> | ||
| <i class="material-icons left">edit</i>Editer | ||
| </a> | ||
| {% endif %} |
There was a problem hiding this comment.
La route bucket_edit est bien réservée au rôle ROLE_SHIFT_MANAGER, mais est-ce le bon test pour autant ?
There was a problem hiding this comment.
On va dire que oui. A voir si un admin peut bien y accéder également. Je ne sais pas si les droits sont bien gérés de ce point de vue. Raphaël semblait m'avoir dit un jour que ce n'était pas trop le cas :)
…our de la modale.
…seulement sur /booking/admin, pas sur /schedule).
|
Globalement OK |
njean42
deleted the
1121-modale-pour-les-créneaux-sur-la-page-planning
branch
Closes #1121
Dans cette première tentative, je réutilise la modale
app/Resources/views/booking/_partial/shift.html.twigen la paramétrant pour la rendre compatible avec le contexte de la page Planning.Je rajoute quelques commentaires dans les changements de code pour montrer les endroits où je ne suis pas sûr de moi − c'est ma première PR ici après tout. 🙂