Projet

Général

Profil

Development #44803

sur les erreurs de ligne d'import CSV, afficher l'erreur

Ajouté par Frédéric Péters il y a presque 4 ans. Mis à jour il y a presque 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
03 juillet 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Elle est actuellement reprise en "title" et ça la rend en pratique invisible pour les gens. (ex: #44801).


Fichiers

0001-manager-show-user-import-errors-inline-44803.patch (5,89 ko) 0001-manager-show-user-import-errors-inline-44803.patch Benjamin Dauvergne, 14 juillet 2020 12:13
csv_import_errors_inline.png (34,7 ko) csv_import_errors_inline.png Paul Marillonnet, 16 juillet 2020 15:13
csv_import_row_errors_inline.png (21,4 ko) csv_import_row_errors_inline.png Paul Marillonnet, 16 juillet 2020 15:31

Révisions associées

Révision eb3caba2 (diff)
Ajouté par Benjamin Dauvergne il y a presque 4 ans

manager: show user import errors inline (#44803)

Historique

#1

Mis à jour par Benjamin Dauvergne il y a presque 4 ans

  • Assigné à mis à Benjamin Dauvergne
#2

Mis à jour par Benjamin Dauvergne il y a presque 4 ans

#3

Mis à jour par Paul Marillonnet il y a presque 4 ans

T'es sûr de ton coup pour le

<td colspan="{{ row|length|add:"1" }}">
?
À vue de nez, on a déjà <td></td> avant et après, je suis pas certain que le compte soit bon.

J'ai essayé de taper n'importe quoi dans le csv pour provoquer une erreur de ligne et vérifier par moi même, mais je n'y arrive pas, ça me jette dehors avant la simulation de l'import :)

Pour la partie erreur de cellule, le rendu fonctionne, cf capture.

#4

Mis à jour par Benjamin Dauvergne il y a presque 4 ans

Paul Marillonnet a écrit :

T'es sûr de ton coup pour le [...] ?
À vue de nez, on a déjà <td></td> avant et après, je suis pas certain que le compte soit bon.

Non j'ai rajouté le deuxième td après, j'ai viré le |add:"1", branche à jour.

J'ai essayé de taper n'importe quoi dans le csv pour provoquer une erreur de ligne et vérifier par moi même, mais je n'y arrive pas, ça me jette dehors avant la simulation de l'import :)

Il faut au minimum que les entêtes soient corrects et le CSV bien formé.

#5

Mis à jour par Paul Marillonnet il y a presque 4 ans

Yep, cette fois-ci c'est bon, cf nouvelle capture.

#6

Mis à jour par Benjamin Dauvergne il y a presque 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit eb3caba25a8d90c81a85403bdc564e4d29217cde
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Mon Jul 13 19:41:34 2020 +0200

    manager: show user import errors inline (#44803)
#7

Mis à jour par Frédéric Péters il y a presque 4 ans

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

Formats disponibles : Atom PDF