search

mviewer / mviewerstudio

mviewer generator
GNU General Public License v3.0
15 stars 25 forks source link

MEG - VSR enregistrement si clic thématique #240

Closed lecault closed 1 year ago

lecault commented 1 year ago

Scénario :

  1. J'ouvre un projet existant
  2. Je vais sur thématiques et données
  3. Je vais sur le menu d'édition de thématique
  4. Je ne fais rien je clic sur enregistrer
  5. Une nouvelle version est créée sur le serveur

Ce n'est pas bloquant mais ça surcharge le nombre de version. Pourquoi le comportement a évolué ?

Gaetanbrl commented 1 year ago

j'ai rajouté un enregistrement au clic sur "enregistrer". Il permet de réaliser une sauvegarde intermédiaire des thématiques et des templates. Inconvénient, le clic sur enregistrer est possible alors qu'aucune modification n'a encore été réalisée. La question se pose surtout sur l'utilité d'un bouton enregistrer et annuler alors qu'aucune action n'a été réalisée.

Je crois que c'est le comportement historique d'avoir 2 boutons ?

Gaetanbrl commented 1 year ago

Mais il est nouveau (depuis hier) d'avoir une action d'enregistrement. le mieux serait de ne pas en avoir si on a rien modifié, mais je crois qu'on avait déjà abordé ce sujet, il avait été considéré comme une évolution.

Gaetanbrl commented 1 year ago

Pour alimenter la discussion, Git détecte toujours une modification en raison de la date qui change à la sauvegarde, donc le backend détecte toujours une modification. Je peux vérifier mais si c'est le seul élément qui change quand on ouvre et clic sur "sauvegarder" sans rien changer, alors la date peut être modifiée dans le backend, ce qui pourrait éviter un commit inutile.

lecault commented 1 year ago

Le fait d'avoir un enregistrement par clic sur enregistrer une thématique va poluer le recensement des versions.

Je me suis fait la réflexion hier qu'en fait le bouton devrait s'appeler Valider plutôt qu'enregistrer. Pour ma part, je souhaite revenir au comportement précédent : enregistrement si clic sur enregistrer ou au premier changement de page.

Gaetanbrl commented 1 year ago

@lecault tu n'as pas vraiment répondu à ma question, mais je viens de pousser un autre comportement. Ce que j'ai testé me convient et permet de ne pas réaliser de nouvelle version si on a pas de modification réalisée (la différence est non nulle).

Est-ce que tu peux tester à nouveau et me dire si pour toi ce comportement est mieux ou moins bien ?

Gaetanbrl commented 1 year ago

Je me suis fait la réflexion hier qu'en fait le bouton devrait s'appeler Valider plutôt qu'enregistrer.

Tu nous avais pas demandé d'harmoniser tous les boutons ? C'était Valider ou Enregistrer qu'on devait mettre je ne sais plus ?

lecault commented 1 year ago

@lecault tu n'as pas vraiment répondu à ma question, mais je viens de pousser un autre comportement. Ce que j'ai testé me convient et permet de ne pas réaliser de nouvelle version si on a pas de modification réalisée (la différence est non nulle).

Oui tu as bien répondu à ma demande d'harmonisation je te confirme, c'est juste une réflexion à froid qui ne rentre pas dans le cadre de la VSR. Enregistrer => enregistre l'application. Valider => valide la page courante.

lecault commented 1 year ago

J'ai relevé un soucis si on a plusieurs thématiques, le comportement ne fonctionne que sur la dernière thématique. Exemple ici : image

Cas 1 :

Cas 2 :

Cas 3 :

Cas 4 :

Cas 5 :

Pour moi mineur.

Gaetanbrl commented 1 year ago

je viens de voir ton retour Loïc (si possible, bien penser à mettre à jour le tableau projet dans la bonne colonne)

Gaetanbrl commented 1 year ago

@lecault a détailler pour chaque étape si possible :

je fais une modif

Quelle modification ?

aucune modif

Tu cliques sur "Enregistrer" ou "annuler" ?

Gaetanbrl commented 1 year ago

Testé suite au message de @lecault

Je créé 2 thématiques :

image

Cas 1 :

Cas 2 :

Cas 3 :

Cas 4 :

Cas 5 :

Où est le comportement qui n'est pas correspondant à tes attentes @lecault ?

lecault commented 1 year ago

Effectivement je ne constate plus de soucis. c'est bon pour moi.