Development #54125
paramètres : renommer "Import" en "Import initial" / "Initialisation" / autre (?)
0%
Description
Noter en gros dessus que c'est initial.
Je serais même pour totalement le retirer dès qu'il y a un formulaire/workflow/modèle de fiches; avis contraire à exprimer.
Fichiers
Révisions associées
Historique
Mis à jour par Emmanuel Cazenave il y a presque 3 ans
Je m'apprêtais à écrire que c'est quand même bien pratique ne local pour reproduire des situations mais je me rends compte que c'est dispo via la CLI, donc je suis tout à fait pour retirer complètement cette bombinette de l'interface oueb quand il y a déjà des données.
Mis à jour par Stéphane Laget il y a presque 3 ans
De mon point de vue une alerte visuelle devrait être suffisante.
C'est un peu dommage de limiter cela à un import initial, ce serait possible à terme de fonctionner avec des slugs plutôt que qu'avec des identifiants, et donc de gérer des écrasements et des créations ?
Dans l'intervalle, la désactivation de l'import devrait donc se faire s'il n'y a pas de modèles/Wfs/etc existants, et donc supposerait de supprimer manuellement l'existant pour ré-utiliser les imports.
Et surtout ne pas bloquer tous les types d'imports si un élément existe dans un autre type d'objets.
Par exemple ne pas bloquer l'import des modèles de mails, si on a déjà des formulaires mais si aucun modèle de mail n'existe encore.
Mis à jour par Frédéric Péters il y a presque 3 ans
La direction par rapport à un import "multiple" c'est le plan "applification", c'est-à-dire la possibilité de regrouper une série de formulaires/fiches/worflows/etc. de les exporter et importer.
Mis à jour par Stéphane Laget il y a presque 3 ans
Oui le plan applification apportera des réponses à ma remarque précédente.
Mais cet import "multiple" est déjà en place.
Lorsque je passe par l'import (dans paramètres), je peux déjà importer des objets de différentes natures. Mon commentaire c'était juste pour demander à ne pas tout bloquer s'il y a déjà des datas dans un type d'objet.
Mis à jour par Frédéric Péters il y a presque 3 ans
Oui ok j'ai noté "De mon point de vue une alerte visuelle devrait être suffisante.".
Mis à jour par Frédéric Péters il y a presque 3 ans
- Fichier 0001-settings-add-a-warning-on-import-page-if-there-s-con.patch 0001-settings-add-a-warning-on-import-page-if-there-s-con.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Voilà juste une info qu'on espérera lue.
Mis à jour par Emmanuel Cazenave il y a presque 3 ans
On rajouterait pas dans la condition d'affichage du warning une vérification sur les blocs de champs, data source, appels webservices ?
Mis à jour par Frédéric Péters il y a presque 3 ans
J'ai hésité mais au final j'ai préféré me limiter à ceux qui ont des gros effets sur les données en base (et aussi en pratique 1/ ça couvre les situations réelles, 2/ on pourrait avoir des sources de données ou appels webservice provisionnés automatiquement dans une mécanique de rayonnement, donc je voulais éviter des blocages ici).
Mis à jour par Emmanuel Cazenave il y a presque 3 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a presque 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
- Assigné à mis à Frédéric Péters
commit bc887791d8ef19d69bd1e3eaeeb0cc1f730ac103 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Mon May 31 09:44:38 2021 +0200 settings: add a warning on import page if there's content already (#54125)
Mis à jour par Frédéric Péters il y a presque 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
settings: add a warning on import page if there's content already (#54125)