Associer deux RM à la même phase

KiDayz commented 3 years ago

J'attendais la résolution de la PR pour mettre le sujet sur la table : avec l'utilisation de Jeyser en prod, on s'est rendu compte que deux RM ne pouvaient pas être liés à une même phase. Cela pose problème dans le cas où Deux réalisateurs travaillent sur la même phase. Par exemple : Phase 1 : réalisation d'un fichier de CAO - 6 JEH à 300€ RM1 - Etudiant 1 - Phase 1 3 JEHs RM2 - ETudiant 2 - Phase 2 3JEHs

Au vu du code, c'est une restriction voulue dans le développement. Je me demandais la raison ? Par exemple sur Beequick, on peut associer deux RM à une même phase. (il y a même une jolie GUI pour cocher le nombre de JEH )

Qu'en pensez-vous ? Seriez-vous ouvert au développement d'une feature comme celle-ci et donc à un changement de BDD ?

KiDayz commented 3 years ago

Hello, je vais commencer à travailler sur un nouveau patch cette semaine, ça pourrait m'aider si on pouvait discuter un peu de ce sujet, je me permets de relancer du coup :)

KiDayz commented 3 years ago

Ce que je propose : Créer une nouvelle classe JEH (une JEH étant forcément lié à une phase) Associer dans un RM d'abord les phases puis les JEH sur le modèle du screen précédent

KiDayz commented 3 years ago

Donc on aura le RM qui sera en relation ManyToMany avec Phase et JEH qui sera en relation ManyToOne avec un RM et ManyToOne avec un Etudiant

Stoakes commented 3 years ago

Pas de souçis avec cette modification, à 2 conditions:

l'UI doit être claire et simple d'utilisation
Jeyser ne doit pas permettre de faire des erreurs sur la répartition des JEH. Comme sauvegarder des situations incorrectes, ne pas répartir tous les JEH, mettre plus de RM sur une phase que le nombre de JEH ... (et certainement bien d'autres)

Enfin, il faut que cette modification soit rétro compatible le plus possible. Est ce que l'objet RepartitionJEH ne pourrait pas faire l'affaire ?

n7consulting / Incipio

Associer deux RM à la même phase #345