Projet

Général

Profil

Development #58217

Pouvoir complètement désactiver autosave pour un formulaire en soumission BO

Ajouté par Benjamin Dauvergne il y a plus de 2 ans. Mis à jour il y a plus de 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
27 octobre 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Ça fait suite à mes deux tickets sur autosave.

Il me semble que pour certains usages purement CRUD où le BO se retrouve fortement chargé par beaucoup d'activité et ou de toute façon l'agent va finir la saisie ou ne jamais revenir dessus, la fonctionnalité autosave (voir les brouillons) n'a pas beaucoup d'utilité et pourrait être désactivé complètement dans tous les cas.


Fichiers

Révisions associées

Révision fbe9dcca (diff)
Ajouté par Frédéric Péters il y a plus de 2 ans

backoffice: disable autosave for backoffice submission (#58217)

Historique

#2

Mis à jour par Frédéric Péters il y a plus de 2 ans

Il y a "pour certains usages" et "dans tous les cas" et ça me fait hésiter sur l'intention.

En soit, si côté fonctionnel c'est ok, ça m'irait tout à fait de totalement désactiver l'autosave en backoffice.

#4

Mis à jour par Benjamin Dauvergne il y a plus de 2 ans

Frédéric Péters a écrit :

Il y a "pour certains usages" et "dans tous les cas" et ça me fait hésiter sur l'intention.

J'essaie d'être prudent, le niveau de preuve demandé me semble assez élevé pour demander un changement des fois. En vrai je trouve ça tout à fait inutile pour les agents dans 99% des cas, je pense que le bénéfice ne vaut pas les risques que ça amène jusqu'à ce qu'on puisse garantir à 100% l'absence de problème.

Je suis à peu près du même avis en front, mais coté fonctionnel c'est beaucoup moins partagé alors on va plutôt continuer dans les évolutions pour limiter les soucis.

#5

Mis à jour par Frédéric Péters il y a plus de 2 ans

  • Statut changé de Nouveau à En cours
  • Assigné à mis à Frédéric Péters

dans 99% des cas

Option prise ainsi de désactiver par défaut, et feature flag dans site-options.cfg si jamais le 1% se révélait exister. (et si bilan dans un an il n'y en a pas, on pourra retirer pour de vrai).

(ça suit #28309 sur lequel j'avais posté un patch, sans lien avec l'activité récente).

#6

Mis à jour par Frédéric Péters il y a plus de 2 ans

#7

Mis à jour par Thomas Noël il y a plus de 2 ans

  • Statut changé de Solution proposée à Solution validée

Dans test_backoffice_parallel_submission tu poses manuellement dans le test le backoffice-autosave = true, mais je pense que la fixture "autosave" le fait déjà pour toi non ?

En dehors de ce détail, go.

#8

Mis à jour par Frédéric Péters il y a plus de 2 ans

  • Statut changé de Solution validée à Résolu (à déployer)

En effet je suis en cours de route parti sur la fixture et j'ai oublié de nettoyer ça, fait.

commit fbe9dcca69f2a2d3a9051035a3b7604c0b4845ee
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Sat Oct 30 15:16:55 2021 +0200

    backoffice: disable autosave for backoffice submission (#58217)
#9

Mis à jour par Frédéric Péters il y a plus de 2 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF