Room reservation by imperosol · Pull Request #1092 · ae-utbm/sith

imperosol · 2025-04-21T15:31:54Z

Fonctionnalité pour permettre aux clubs de gérer des salles que les gens peuvent réserver.

Aperçu

Le coeur de la fonctionnalité est le planning de réservation :

Ce planning est accessible depuis les liens sur la page d'accueil :

Chaque salle est liée à un site (Belfort, Sev ou Montbé) et gérée par un club.

Un club peut gérer ses salles depuis ses outils

Les salles sont réservables. Une réservation possède un auteur (la personne qui a fait la réservation), une salle, une heure de début et une heure de fin. Comme on peut le voir plus haut, il y a marqué en-dessous du planning qu'on peut créer une réservation en sélectionnant une plage horaire. Lorsqu'on le fait, un formulaire apparait avec les champs appropriés pré-remplis.

Si on a les permissions requises, on peut également déplacer les réservations directement par glisser-déposer sur le planning.

Modération et permission

Il n'y a pas (encore) de système de modération des événements. La PR est déjà assez grosse comme ça, je préfère passer une gestion plus fine des permissions dans une autre mise-à-jour plus tard.

Pour le moment, les réservations sont créables avec la permission globale reservation.add_reservationslot, qui sera attribuée aux respos sites (et éventuellement au bureau AE dans son ensemble aussi).
Les réservations sont modifiables avec la permission reservation.change_reservationslot, attribuée aux respos site.

Même avec les permissions requises, les réservations passées ne sont ni créables, ni modifiables. Une réservation dont l'heure de début se trouve avant le moment présent est donc gelée.

klmp200

Petit début de review parce que j'ai un peu de temps, je m'y remet plus tard

klmp200 · 2025-08-15T16:30:53Z

+   */
+  async fetchResources() {
+    const res = await reservableroomFetchRooms();
+    return res.data.map((i) => Object.assign(i, { title: i.name, group: i.location }));


Vu que la majorité des gens vont pas avoir le droit de voir cette page, ce serait bien de faire qu'il n'y ai pas le lien de la page pour tous les gens connectés, sinon les gens voient un truc pété comme ça quand l'api renvoi une 403

C'est déjà le cas. Tu ne vois le lien et tu n'as le droit d'accéder à la vue que si tu a la permission de voir des réservations. Tu as peut-être oublié de générer les données de test.

Du coup, tu peux avoir le droit de voir la page mais pas forcément le droit de voir les salles, c'est étrange

C'est même pire que ça. J'ai pris krophil qui est président troll (qui possède la jolly), il a la permission de réservation et pourtant on voit aucune salle dans le tableau.

Il y a un truc que je comprend pas

Je vais regarder ça

klmp200 · 2025-08-15T16:43:59Z

              <i class="fa-solid fa-magnifying-glass fa-xl"></i>
              <a href="{{ url("matmat:search_clear") }}">{% trans %}Matmatronch{% endtrans %}</a>
            </li>
+            {% if user.has_perm("reservation.view_reservationslot") %}


Mhh, ça a pas vraiment l'air de fonctionner cette histoire
J'ai connecté plusieurs users et on arrive juste sur une page pétée parce que l'api renvoie une 403 dessus

Perso, j'ai bien ça

Ok, j'ai trouvé, d'après le populate, tous les cotisants ont cette perm.
En prod ce sera le cas ou pas ?

Oui
Après, c'est comme toutes les autres perms de django, on peut choisir directement dans l'interface admin quels groupes ont quelles perms

klmp200 · 2026-03-27T18:08:40Z

              <i class="fa-solid fa-magnifying-glass fa-xl"></i>
              <a href="{{ url("matmat:search_clear") }}">{% trans %}Matmatronch{% endtrans %}</a>
            </li>
+            {% if user.has_perm("reservation.view_reservationslot") %}


Ok, j'ai trouvé, d'après le populate, tous les cotisants ont cette perm.
En prod ce sera le cas ou pas ?

klmp200 · 2026-03-27T18:13:38Z

+        )
+        if self.id is not None:
+            overlapping = overlapping.exclude(id=self.id)
+        if overlapping.exists():


Alors, j'ai pas spécialement try hard mais j'ai l'impression qu'il y a un petit soucis là. Si tu met EXACTEMENT les mêmes heures, il ne voit pas ça comme overlapping

C'est un problème de lt/gt au lieu de lte/gte

klmp200 · 2026-03-27T18:17:41Z

+   */
+  async fetchResources() {
+    const res = await reservableroomFetchRooms();
+    return res.data.map((i) => Object.assign(i, { title: i.name, group: i.location }));


Du coup, tu peux avoir le droit de voir la page mais pas forcément le droit de voir les salles, c'est étrange

klmp200 · 2026-03-27T18:26:51Z

+      resources: this.fetchResources,
+      events: this.fetchEvents,
+      select: this.selectFreeSlot,
+      selectOverlap: false,


On peut pas déplacer entre les salles la réservation, il y a une option pour ça ?

klmp200 · 2026-03-27T18:29:18Z

+      {% endfor %}
+    </div>
+  {% endif %}
+  <form


Utilisation intéressante du formulaire.
Par contre, si le but c'était juste d'ajouter des attributs, on peut le faire directement dans un formulaire django https://www.geeksforgeeks.org/python/how-to-add-html-attributes-to-input-fields-in-django-forms/#methods-to-add-html-attributes

Les attributs Alpine, je préfère les mettre dans le template. C'est pas juste un placeholder ou un autre attribut qui marche bien tout seul. L'attribut Alpine, s'il est là, c'est pour interagir avec autre chose. Si on définit les variables Alpine à plusieurs endroits différents, ça devient difficile à suivre.

klmp200 · 2026-03-27T18:36:09Z

+        verbose_name=_("reserved room"),
+    )
+    author = models.ForeignKey(User, on_delete=models.CASCADE, verbose_name=_("author"))
+    comment = models.TextField(_("comment"), blank=True, default="")


Le commentaire, j'ai trouvé aucun moyen de le voir, même pas un petit tooltip, rien :/

klmp200 · 2026-03-27T18:39:31Z

+      href="{{ url("reservation:room_create") }}?club={{ object.id }}"
+      class="btn btn-blue"
+    >
+      {% trans %}Add a room{% endtrans %}


Avec l'utilisateur krophil, président club.

Ptet un ptit check pour retirer le bouton si t'as pas les droits pour ajouter une salle

klmp200 · 2026-03-27T18:41:38Z

+        _("site"),
+        blank=True,
+        choices=[
+            ("BELFORT", "Belfort"),


Ptet un peu hardcodé quand même :/
Ça devrais pas changer mais ptet au moins mettre ça dans les settings, non ?

klmp200 · 2026-03-27T18:42:38Z

+    path("room/<int:room_id>/delete", RoomDeleteView.as_view(), name="room_delete"),
+    path(
+        "fragment/reservation", ReservationFragment.as_view(), name="make_reservation"
+    ),


Une page qui liste toutes les salles ce serait pas de refus, c'est vraiment difficile de s'y retrouver.
Mais ça peut être fait plus tard

La page qui liste toutes les salles, c'est la page des réservations

klmp200 · 2026-03-27T18:43:10Z

+  private locale = "en";
+  private canEditSlot = false;
+  private canBookSlot = false;
+  private canDeleteSlot = false;


J'ai toujours pas trouvé comment delete un slot :/

imperosol force-pushed the room-reservation branch 2 times, most recently from 926122e to 3ebd4b1 Compare April 23, 2025 12:23

imperosol force-pushed the room-reservation branch from 3ebd4b1 to 7eb1fab Compare May 16, 2025 18:31

imperosol force-pushed the room-reservation branch 2 times, most recently from b99fbc0 to ddf3382 Compare June 5, 2025 10:52

imperosol force-pushed the room-reservation branch 5 times, most recently from be1f35b to 03a04fc Compare June 18, 2025 14:14

imperosol force-pushed the room-reservation branch 2 times, most recently from 438b632 to cc5ab90 Compare June 21, 2025 21:40

imperosol force-pushed the room-reservation branch from cc5ab90 to 6d21ecb Compare June 27, 2025 11:50

imperosol marked this pull request as ready for review June 28, 2025 11:25

auto-assign Bot assigned imperosol Jun 28, 2025

imperosol force-pushed the room-reservation branch from 6d21ecb to 5b29250 Compare June 28, 2025 11:37

klmp200 reviewed Jun 29, 2025

View reviewed changes

Comment thread core/static/core/style.scss Outdated

Comment thread package.json Outdated

Comment thread reservation/static/bundled/reservation/components/room-scheduler-index.ts

imperosol force-pushed the room-reservation branch 4 times, most recently from 04e2e00 to 0baaf69 Compare June 30, 2025 14:16

klmp200 reviewed Aug 15, 2025

View reviewed changes

imperosol force-pushed the room-reservation branch from dfb545b to bfc30f4 Compare August 25, 2025 15:20

imperosol force-pushed the room-reservation branch from bfc30f4 to 7655200 Compare September 4, 2025 10:04

imperosol force-pushed the room-reservation branch from 7655200 to 4cc9339 Compare September 13, 2025 09:46

imperosol force-pushed the room-reservation branch 3 times, most recently from b8bf684 to 03cdad5 Compare September 26, 2025 21:31

imperosol requested review from Aonfu and ken-soares October 8, 2025 13:34

imperosol force-pushed the room-reservation branch from 03cdad5 to e5a2d1b Compare October 11, 2025 11:19

imperosol force-pushed the room-reservation branch from e5a2d1b to 21dc955 Compare November 23, 2025 01:03

imperosol force-pushed the room-reservation branch from 21dc955 to 753b219 Compare March 8, 2026 18:26

klmp200 requested changes Mar 27, 2026

View reviewed changes

klmp200 and others added 13 commits May 23, 2026 15:28

Adapt calendar to new tooltip library

28551f8

create reservation models

b6f25bf

generate test data for the reservations

2c23e00

reservable rooms API

c1084f1

fix: FutureDateTime form field

b86ad5e

room management views

e973b01

Room reservations planning

e2a417b

Room reservation form

3f3add0

add translations

6114184

test: ReservationForm

4bbe049

fix: rebase issues

d54e323

test: room and slots creation/edition

2d91526

add feedback when moving reservation slot

f4a70e0

imperosol force-pushed the room-reservation branch from 753b219 to f4a70e0 Compare May 23, 2026 14:14

Uh oh!

Conversation

imperosol commented Apr 21, 2025 • edited by TitouanDor Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Aperçu

Modération et permission

Uh oh!

Uh oh!

Uh oh!

Uh oh!

klmp200 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

imperosol commented Apr 21, 2025 •

edited by TitouanDor

Loading