DISIC / observatoire

🕵️‍♀️ Observatoire de la qualité des démarches en ligne
https://observatoire.numerique.gouv.fr/
GNU Lesser General Public License v3.0
10 stars 4 forks source link

Obs Admin & Porteur: erreur lors de la copie d'une démarche #1064

Open AnthonyBrunelli opened 2 years ago

AnthonyBrunelli commented 2 years ago

Lorsque nous essayons de copier une démarche (pour que les informations soient pré saisies) le message d’alerte suivant s’affiche : “le renommage a échoué”

AnthonyBrunelli commented 2 years ago

image(1)

AnthonyBrunelli commented 2 years ago

Relevé par Joffrey

jrenauxdinum commented 2 years ago
lucaa commented 2 years ago

Je dois vérifier les logs pour comprendre pourquoi la copie ne fonctionne pas mais même si elle fonctionnait, le résultat de cette opération faite comme ça ne serait pas tout à fait valide du pdv du modèle de données (car cette nouvelle démarche de test créée suite à la copie n'aurait pas un id comme toutes les autres démarches...)

L'option est disponible car, pour les admins, aucun travail spécifique n'a été fait pour nettoyer les fonctionnalités de base de la plateforme XWiki qui ne sont pas compatibles avec le modèle de données des démarches.

Pour l'instant je vous propose de créer cette copie à partir des formulaire de création.

jrenauxdinum commented 2 years ago

Mince, donc finalement mieux vaut ne pas utiliser cette fonctionnalité si cela donne des démarches sans ID Est ce qu'on peut supprimer cette fonctionnalité facilement ? En l'état, j'ai l'impression que cela risque de plus créer des problèmes que répondre à des besoins.

lucaa commented 2 years ago

@jrenauxdinum en fait, en cherchant l'erreur je me rends compte que la copie a été créée, ici https://observatoire.numerique.gouv.fr/Demarches/E-social%20des%20arm%C3%A9es%20%3A%20TEST , mais comme tu peux voir elle n'a pas d'ID correct.

Est ce qu'on peut supprimer cette fonctionnalité facilement ?

oui, il faut créer un ticket et le planifier dans la version suivante. Il faudrait aussi vérifier si ce changement impacte tous les rôles (admin dinum, porteur, admin ministeriel).

lucaa commented 2 years ago

Il faudrait aussi vérifier si ce changement impacte tous les rôles (admin dinum, porteur, admin ministeriel).

J'ai vérifié les autres logs de copies échouées, elles sont lancées par des porteurs. Donc c'est une fonctionnalité qui est visible aux porteurs alors qu'elle n'est pas vraiment fonctionnelle ni valide.

jrenauxdinum commented 2 years ago

D'accord, je vais donc voir avec @AnthonyBrunelli comment / quand on peut la supprimer Pour ce qui est de l'impact sur les rôles, c'est une analyse à faire de notre côté ou du votre ?

lucaa commented 2 years ago

J'ai vérifié les autres logs de copies échouées, elles sont lancées par des porteurs.

pour le porteur l'erreur est liée aux droits de création d'une nouvelle démarche et la démarche n'est pas créée.

en cherchant l'erreur je me rends compte que la copie a été créée, ici https://observatoire.numerique.gouv.fr/Demarches/E-social%20des%20arm%C3%A9es%20%3A%20TEST , mais comme tu peux voir elle n'a pas d'ID correct.

pour les admins dinum, la copie de la démarche est créée mais il y a une erreur liée à une initialisation qui doit être faite lors de la création d'une démarche. En tout cas, comme c'est une opération qui n'est pas valide du pdv du modèle des données, il faudrait bloquer ce chemin pour tous les utilisateurs.

lucaa commented 2 years ago

Pour ce qui est de l'impact sur les rôles, c'est une analyse à faire de notre côté ou du votre ?

C'est une analyse fonctionnelle qui est nécessaire: comme pour toute fonctionnalité / modification de fonctionnalité la demande de changement doit être complète, couvrir tous les aspects et tous les rôles qui pourraient être impactés par cette fonctionnalité. Je peux faire aussi cette analyse mais je pense que c'est plus efficace que ça soit faite du côté définition du produit.