entrouvert
/
chrono
14
1
Fork 0
Code Issues Pull Requests 4 Releases Wiki Activity

affichage en backoffice d'un préblocage (#82774) #180

Merged
bdauvergne merged 2 commits from wip/82774-affichage-en-backoffice-d-un-pre into main 2023-11-28 14:48:03 +01:00
Conversation 8 Commits 2 Files Changed 9 +130 -28
bdauvergne commented 2023-11-21 18:32:24 +01:00
Owner
Copy Link
No description provided.
bdauvergne added 2 commits 2023-11-21 18:32:25 +01:00
bdauvergne force-pushed wip/82774-affichage-en-backoffice-d-un-pre from 6a7eefae73 to b81460ebbd 2023-11-21 18:33:10 +01:00 Compare
bdauvergne force-pushed wip/82774-affichage-en-backoffice-d-un-pre from b81460ebbd to 579526df09 2023-11-22 09:50:09 +01:00 Compare
bdauvergne force-pushed wip/82774-affichage-en-backoffice-d-un-pre from 579526df09 to 89c749743e 2023-11-22 16:11:33 +01:00 Compare
bdauvergne force-pushed wip/82774-affichage-en-backoffice-d-un-pre from 89c749743e to ee0197ad91 2023-11-22 16:19:28 +01:00 Compare
bdauvergne force-pushed wip/82774-affichage-en-backoffice-d-un-pre from ee0197ad91 to e539ee3160 2023-11-22 16:19:59 +01:00 Compare
bdauvergne force-pushed wip/82774-affichage-en-backoffice-d-un-pre from e539ee3160 to 20c656393b 2023-11-22 16:54:09 +01:00 Compare
bdauvergne commented 2023-11-22 17:02:36 +01:00
Author
Owner
Copy Link

Le rendu dans les 3 vues agenda rdv et la vue de détail d'un évènement.

Le rendu dans les 3 vues agenda rdv et la vue de détail d'un évènement.
Firefox_Screenshot_2023-11-22T11-47-48.858Z.png
79 KiB
Firefox_Screenshot_2023-11-22T11-50-18.815Z.png
47 KiB
Firefox_Screenshot_2023-11-22T11-49-43.156Z.png
104 KiB
Firefox_Screenshot_2023-11-22T14-40-18.557Z.png
52 KiB
Firefox_Screenshot_2023-11-22T11-47-48.858Z.png Firefox_Screenshot_2023-11-22T11-50-18.815Z.png Firefox_Screenshot_2023-11-22T11-49-43.156Z.png Firefox_Screenshot_2023-11-22T14-40-18.557Z.png
bdauvergne changed title from WIP: affichage en backoffice d'un préblocage (#82774) to affichage en backoffice d'un préblocage (#82774) 2023-11-22 17:03:01 +01:00
nroche reviewed 2023-11-22 17:09:25 +01:00
chrono/manager/static/css/style.scss Outdated
@ -1,4 +1,4 @@
.occupation-bar {
eoccupation-bar {
nroche commented 2023-11-22 17:09:25 +01:00
Owner
Copy Link

(typo ici)

(typo ici)
bdauvergne commented 2023-11-22 18:27:37 +01:00
Author
Owner
Copy Link

Corrigé.

Corrigé.
bdauvergne force-pushed wip/82774-affichage-en-backoffice-d-un-pre from 20c656393b to ac763a6d20 2023-11-22 18:26:01 +01:00 Compare
nroche approved these changes 2023-11-27 20:14:16 +01:00
nroche left a comment
Owner
Copy Link

J'ai l'impression que les templates serait plus lisibles si l'on n'utilisait pas de lien pour afficher les "leases".
Je joins un patch parce que je pense que c'est plus simple ici pour te faire un avis.

J'ai l'impression que les templates serait plus lisibles si l'on n'utilisait pas de lien pour afficher les "leases". Je joins un patch parce que je pense que c'est plus simple ici pour te faire un avis.
0001-manager-do-not-use-link-for-leased-bookings-82774.patch
8.6 KiB
vdeniaud commented 2023-11-28 10:41:47 +01:00
Owner
Copy Link

Vraiment pas fan d'utiliser le même zebra que les périodes d'exception, il aurait fallu montrer ce que ça rend sur les screenshots (en l'état mon imagination me dit que c'est confusant).

(oui cette remarque arrive trop tard, je n'avais pas vu les suggestions d'Anaïs dans le ticket, il aurait fallu dire stop à ce moment là : le ticket c'était vraiment juste avoir un libellé mieux que « Anonyme », si il y avait du CSS à faire ça pouvait être vu dans encore un autre ticket, pris par un spécialiste front)

Autre remarque pendant que j'y suis, le screenshot de la vue évènement est coupé, mais à lire le code je pense que le bouton annuler est présent, je ne sais pas si on veut vraiment ça (et si on veut pas ça peut être vu dans un autre ticket).

Bref tout ça pour dire c'est validé je ne m'oppose pas à la validation.

Vraiment pas fan d'utiliser le même zebra que les périodes d'exception, il aurait fallu montrer ce que ça rend sur les screenshots (en l'état mon imagination me dit que c'est confusant). (oui cette remarque arrive trop tard, je n'avais pas vu les suggestions d'Anaïs dans le ticket, il aurait fallu dire stop à ce moment là : le ticket c'était vraiment juste avoir un libellé mieux que « Anonyme », si il y avait du CSS à faire ça pouvait être vu dans encore un autre ticket, pris par un spécialiste front) Autre remarque pendant que j'y suis, le screenshot de la vue évènement est coupé, mais à lire le code je pense que le bouton annuler est présent, je ne sais pas si on veut vraiment ça (et si on veut pas ça peut être vu dans un autre ticket). Bref tout ça pour dire c'est validé je ne m'oppose pas à la validation.
bdauvergne commented 2023-11-28 12:29:22 +01:00
Author
Owner
Copy Link

Vraiment pas fan d'utiliser le même zebra que les périodes d'exception, il aurait fallu montrer ce que ça rend sur les screenshots (en l'état mon imagination me dit que c'est confusant).

Ça pourra toujours être corrigé ensuite, en l'état des choses on ne va pas être envahi par ces zébras.

(oui cette remarque arrive trop tard, je n'avais pas vu les suggestions d'Anaïs dans le ticket, il aurait fallu dire stop à ce moment là : le ticket c'était vraiment juste avoir un libellé mieux que « Anonyme », si il y avait du CSS à faire ça pouvait être vu dans encore un autre ticket, pris par un spécialiste front)

Je préfère avancer et corriger ensuite, c'est assez mineur de devoir revoir la CSS du fond et les règles sont bien circonscrites à cette feature.

Autre remarque pendant que j'y suis, le screenshot de la vue évènement est coupé, mais à lire le code je pense que le bouton annuler est présent, je ne sais pas si on veut vraiment ça (et si on veut pas ça peut être vu dans un autre ticket).

Tu peux voir dans le patch sur template que le bouton cancel est bien supprimé.

+          <a {% if booking.get_backoffice_url %}href="{{ booking.get_backoffice_url }}"{% endif %}>{% if booking.lease %}{% trans "Currently being booked..." %}{% else %}{{ booking.get_user_block }}, {{ booking.creation_datetime|date:"DATETIME_FORMAT" }}{% endif %}</a>
+          {% if not booking.lease %}
+            {% if not booking.primary_booking %}
+              <a rel="popup" class="delete" href="{% url 'chrono-manager-booking-cancel' pk=agenda.id booking_pk=booking.id %}?next={{ request.path }}">{% trans "Cancel" %}</a>
+            {% else %}
+              <a class="delete disabled" title="{% trans "Can not cancel a secondary booking" %}" href="#">{% trans "Cancel" %}</a>
+            {% endif %}
           {% endif %}
> Vraiment pas fan d'utiliser le même zebra que les périodes d'exception, il aurait fallu montrer ce que ça rend sur les screenshots (en l'état mon imagination me dit que c'est confusant). Ça pourra toujours être corrigé ensuite, en l'état des choses on ne va pas être envahi par ces zébras. > (oui cette remarque arrive trop tard, je n'avais pas vu les suggestions d'Anaïs dans le ticket, il aurait fallu dire stop à ce moment là : le ticket c'était vraiment juste avoir un libellé mieux que « Anonyme », si il y avait du CSS à faire ça pouvait être vu dans encore un autre ticket, pris par un spécialiste front) Je préfère avancer et corriger ensuite, c'est assez mineur de devoir revoir la CSS du fond et les règles sont bien circonscrites à cette feature. > Autre remarque pendant que j'y suis, le screenshot de la vue évènement est coupé, mais à lire le code je pense que le bouton annuler est présent, je ne sais pas si on veut vraiment ça (et si on veut pas ça peut être vu dans un autre ticket). Tu peux voir dans le patch sur template que le bouton cancel est bien supprimé. ```+ <li{% if booking.lease %} class="lease"{% endif %}> + <a {% if booking.get_backoffice_url %}href="{{ booking.get_backoffice_url }}"{% endif %}>{% if booking.lease %}{% trans "Currently being booked..." %}{% else %}{{ booking.get_user_block }}, {{ booking.creation_datetime|date:"DATETIME_FORMAT" }}{% endif %}</a> + {% if not booking.lease %} + {% if not booking.primary_booking %} + <a rel="popup" class="delete" href="{% url 'chrono-manager-booking-cancel' pk=agenda.id booking_pk=booking.id %}?next={{ request.path }}">{% trans "Cancel" %}</a> + {% else %} + <a class="delete disabled" title="{% trans "Can not cancel a secondary booking" %}" href="#">{% trans "Cancel" %}</a> + {% endif %} {% endif %} ```
bdauvergne force-pushed wip/82774-affichage-en-backoffice-d-un-pre from ac763a6d20 to aad10c71ee 2023-11-28 12:32:16 +01:00 Compare
bdauvergne commented 2023-11-28 12:52:39 +01:00
Author
Owner
Copy Link

J'ai intégré le patch de Nico, je vais remettre des screenshots.

J'ai intégré le patch de Nico, je vais remettre des screenshots.
bdauvergne commented 2023-11-28 13:04:53 +01:00
Author
Owner
Copy Link

Avec une période d'exception.

Avec une période d'exception.
Firefox_Screenshot_2023-11-28T12-04-15.240Z.png
76 KiB
Firefox_Screenshot_2023-11-28T12-04-32.084Z.png
100 KiB
Firefox_Screenshot_2023-11-28T12-04-15.240Z.png Firefox_Screenshot_2023-11-28T12-04-32.084Z.png
vdeniaud commented 2023-11-28 13:38:59 +01:00
Owner
Copy Link

Je préfère avancer et corriger ensuite, c'est assez mineur de devoir revoir la CSS du fond et les règles sont bien circonscrites à cette feature.

On est d'accord, merci pour les captures, mettons qu'entre la taille, le fond vert et l'ombre c'est assez différencié des périodes d'exception.

> Je préfère avancer et corriger ensuite, c'est assez mineur de devoir revoir la CSS du fond et les règles sont bien circonscrites à cette feature. On est d'accord, merci pour les captures, mettons qu'entre la taille, le fond vert et l'ombre c'est assez différencié des périodes d'exception.
bdauvergne merged commit aad10c71ee into main 2023-11-28 14:48:03 +01:00
bdauvergne deleted branch wip/82774-affichage-en-backoffice-d-un-pre 2023-11-28 14:48:03 +01:00
nroche reviewed 2023-11-29 09:22:46 +01:00
chrono/manager/templates/chrono/manager_event_detail_fragment.html
@ -54,3 +58,3 @@
<ul class="objects-list single-links">
{% for booking in waiting %}
<li><a {% if booking.get_backoffice_url %}href="{{ booking.get_backoffice_url }}"{% endif %}>{{ booking.get_user_block }}, {{ booking.creation_datetime|date:"DATETIME_FORMAT" }}</a></li>
<li{% if booking.lease %} class="lease"{% endif %}><a {% if booking.get_backoffice_url %}href="{{ booking.get_backoffice_url }}"{% endif %}>{% if booking.lease %}{% trans "Currently being booked..." %}{% else %}{{ booking.get_user_block }}, {{ booking.creation_datetime|date:"DATETIME_FORMAT" }}{% endif %}</a></li>
nroche commented 2023-11-29 09:22:46 +01:00
Owner
Copy Link

J'ai oublié de retirer le lien ici dans le précédent patch.

J'ai oublié de retirer le lien ici dans le précédent patch.
nroche commented 2023-11-29 09:24:52 +01:00
Owner
Copy Link

Idem, tant qu'à faire je te passe le patch et la capture d'écran pour la liste d'attente sur les événements.

Idem, tant qu'à faire je te passe le patch et la capture d'écran pour la liste d'attente sur les événements.
0001-manager-do-not-use-link-for-leased-bookings-82774.patch
1.8 KiB
Screenshot 2023-11-29 at 09-20-08 Test des événements Agendas.png
28 KiB
Screenshot 2023-11-29 at 09-20-08 Test des événements Agendas.png
Sign in to join this conversation.
Reviewers
No reviewers
nroche
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/chrono#180
No description provided.
Delete Branch "wip/82774-affichage-en-backoffice-d-un-pre"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?