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

Shift : permettre à un admin de donner une raison lors de l'annulation #713

Merged
merged 2 commits into from
Jan 26, 2023

Conversation

raphodn
Copy link
Member

@raphodn raphodn commented Jan 24, 2023

Quoi ?

Coté Admin, lors de la "libération" d'un créneau à venir, afficher une modale de confirmation / justification éventuelle. (modale similaire à celle existante coté profil)

Quid de la libération de créneau dans la page "Gérer les créneaux" ? pas possible actuellement / facilement, car les créneaux s'affichent déjà dans une modale, je n'ai pas trouvé comment afficher une 2ème modale par dessus.

Pourquoi ?

Cette justification s'affichera dans "ShiftFreeLog"

Capture d'écran

Screenshot from 2023-01-24 16-00-54

@raphodn raphodn self-assigned this Jan 24, 2023
@raphodn raphodn requested a review from petitalb January 24, 2023 15:03
@raphodn raphodn linked an issue Jan 24, 2023 that may be closed by this pull request
@@ -50,7 +50,8 @@ It use the materialize modal class https://materializecss.com/modals.html
{% endif %}
{{ form_end(shift_validate_invalidate_forms[shift.id]) }}
{% endif %}
{{ form_start(shift_free_forms[shift.id], {'attr': { 'style': 'display:inline;'}}) }}
{{ form_start(shift_free_forms[shift.id], {'attr': { 'style': 'display:inline;' }}) }}
{% do shift_free_forms[shift.id].reason.setRendered() %} <!-- hidden -->
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

je connaissais pas cette syntaxe, tu as un pointeur ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@raphodn raphodn force-pushed the raphodn/log-free-shift-4 branch from a39e6a3 to df6cb58 Compare January 26, 2023 12:21
@raphodn raphodn changed the base branch from raphodn/log-free-shift-3 to master January 26, 2023 12:22
@raphodn raphodn merged commit 8cf2283 into master Jan 26, 2023
@raphodn raphodn deleted the raphodn/log-free-shift-4 branch January 26, 2023 12:22
quot17 pushed a commit to quot17/gestion-compte that referenced this pull request Mar 28, 2023
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.

Garder une trâce des créneaux annulés par un membre
2 participants