Projet

Général

Profil

Development #14040

export CSV de la liste des transactions

Ajouté par Frédéric Péters il y a plus de 7 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Haut
Assigné à:
Jean-Baptiste Jaillet
Version cible:
-
Début:
21 novembre 2016
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Dans la page listant les transactions, avoir un lien permettant de télécharger le tableau sous forme de fichier CSV.


Fichiers

0001-lingo-add-csv-export-link-for-transactions-14040.patch (6,27 ko) 0001-lingo-add-csv-export-link-for-transactions-14040.patch Jean-Baptiste Jaillet, 01 décembre 2016 11:51
0001-lingo-add-csv-export-link-for-transactions-14040.patch (6,26 ko) 0001-lingo-add-csv-export-link-for-transactions-14040.patch Jean-Baptiste Jaillet, 01 décembre 2016 18:18
0001-lingo-add-csv-export-link-for-transactions-14040.patch (6,3 ko) 0001-lingo-add-csv-export-link-for-transactions-14040.patch Jean-Baptiste Jaillet, 06 décembre 2016 16:53
0001-lingo-add-csv-export-link-for-transactions-14040.patch (7,44 ko) 0001-lingo-add-csv-export-link-for-transactions-14040.patch Jean-Baptiste Jaillet, 06 décembre 2016 18:08
0001-lingo-add-csv-export-link-for-transactions-14040.patch (7,45 ko) 0001-lingo-add-csv-export-link-for-transactions-14040.patch Jean-Baptiste Jaillet, 06 décembre 2016 18:11
0001-lingo-add-csv-export-link-for-transactions-14040.patch (7,39 ko) 0001-lingo-add-csv-export-link-for-transactions-14040.patch Jean-Baptiste Jaillet, 07 décembre 2016 17:51
0001-lingo-add-csv-export-link-for-transactions-14040.patch (7,39 ko) 0001-lingo-add-csv-export-link-for-transactions-14040.patch Jean-Baptiste Jaillet, 07 décembre 2016 17:57
0001-lingo-add-csv-export-link-for-transactions-14040.patch (7,36 ko) 0001-lingo-add-csv-export-link-for-transactions-14040.patch Jean-Baptiste Jaillet, 07 décembre 2016 17:59
0001-lingo-add-csv-export-link-for-transactions-14040.patch (7,35 ko) 0001-lingo-add-csv-export-link-for-transactions-14040.patch Jean-Baptiste Jaillet, 07 décembre 2016 18:34

Demandes liées

Lié à Combo - Development #14023: Attacher un BasketItem local aux transactions (sur des factures, remote_items)Fermé20 novembre 2016

Actions

Révisions associées

Révision 48595b46 (diff)
Ajouté par Jean-Baptiste Jaillet il y a plus de 7 ans

lingo: add csv export link for transactions (#14040)

Historique

#2

Mis à jour par Jean-Baptiste Jaillet il y a plus de 7 ans

Voilà avec le patch.

#3

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 7 ans

order_id, bank_transaction_id sont déjà des chaînes, pas besoin d'en faire des str.
Pour le user tu peux appeler user.get_full_name.

Pour le formatage de la date tu peux utiliser la constante DATETIME_FORMAT (https://docs.djangoproject.com/en/1.10/ref/settings/#std:setting-DATETIME_FORMAT), ça a l'avantage d'être localisé en fonction de la langue.

#4

Mis à jour par Jean-Baptiste Jaillet il y a plus de 7 ans

Changé et comme vu avec Serghei, je garde la date comme ça.

#5

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

  • Priorité changé de Normal à Haut
#6

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

Changé et comme vu avec Serghei, je garde la date comme ça.

(utile de donner un résumé de l'argument)

transaction.start_date.strftime('%d-%m-%Y %H:%M'),

Soit on met un format français avec des / comme séparateurs, soit un format international avec l'ordre %Y-%m-%d.

Dans le ticket lié, il est noté qu'il faudrait aussi le numéro de facture; ça demande #14023.

assert float(row4) float(trans2.amount)

Non, on ne compare des float() avec des . → Decimal.

#7

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

  • Lié à Development #14023: Attacher un BasketItem local aux transactions (sur des factures, remote_items) ajouté
#8

Mis à jour par Jean-Baptiste Jaillet il y a plus de 7 ans

Ok pour le format des dates, je voulais garder l'affichage du front.
La numéro de facture c'est le bank_transaction_id c'est bien ça?

#9

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

Ok pour le format des dates, je voulais garder l'affichage du front.

Si en front c'est ce mauvais format qui est affiché, il faut faire un ticket.

La numéro de facture c'est le bank_transaction_id c'est bien ça?

Non, c'est RemoteItem.display_id, qui devrait être (#14023) copié dans un BasketItem.subject.

(et ainsi ça apparaitra dans le transaction.items.all() et ce ticket pourrait ainsi arriver indépendamment).

#10

Mis à jour par Jean-Baptiste Jaillet il y a plus de 7 ans

OK.
Je comprenais pas trop pourquoi on avait items et remote_items dans transaction. Je commence à comprendre un peu mieux.
Du coup (si j'ai bien compris #14023), pour chaque remote_items, on va créer un BasketItem et dans subject mettre le display_id ? =>
  • Est ce que du coup le champs remote_items de transaction à encore du sens, ou on garde les deux ? Ou est ce que ces 'items locaux' ne servent qu'à garder le display_id ?
  • Si on à plusieurs items sur la transactions, quel display_id affiche-t-on en numéro de facture ?
  • Est-ce qu'on modifie transaction_list.html pour afficher se display_id ?
#11

Mis à jour par Jean-Baptiste Jaillet il y a plus de 7 ans

Du coup en revoyant avec Thomas, le patch devrait être bon une fois que la modif du #14023 sera faite (c'est en cours) : on aura les numéros de facture dans les subject de items locaux créés.
Je mets avec les corrections de format de date et de float => Decimal

#12

Mis à jour par Jean-Baptiste Jaillet il y a plus de 7 ans

Re up avec la petite distinction qu'une transaction peut être anonyme.
Avec le patch

#13

Mis à jour par Jean-Baptiste Jaillet il y a plus de 7 ans

Sans faute à Anonymous... (je me suis pris à soupirer aussi).

#14

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

pour la date, je serai partisan de strftime('%Y-%m-%d %H:%M:%S') (voire strftime('%Y-%m-%dT%H:%M:%S.%f') mais bon c'est sans doute un peu brutal)

#15

Mis à jour par Jean-Baptiste Jaillet il y a plus de 7 ans

Hop fait. C'est vrai qu'au moins c'est international.
Et j'ai modifié les petites fautes 'de code style' aussi (import csv, espace en trop..)

#16

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

Jean-Baptiste Jaillet a écrit :

Et j'ai modifié les petites fautes 'de code style' aussi (import csv, espace en trop..)

allez, encore un effort ;)

#18

Mis à jour par Jean-Baptiste Jaillet il y a plus de 7 ans

J'ai enlevé l'import du settings dans manager_views que j'avais ajouté et qui ne sert à rien.

#19

Mis à jour par Jean-Baptiste Jaillet il y a plus de 7 ans

Et j'ajoute la ligne vide entre la license et l'import csv

#20

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

Et à cet instant où j'allais écrire Ack, relecture du ticket de base #14188: « pourrait prendre la forme d'un export CSV du journal des paiements (ou autre) avec une sélection de dates (de mois). »

Et ça me semble assez évident qu'il faut ça, à voir comment on le présente. Je profiterai de ce filtre pour permettre d'avoir un CSV de toutes les transactions, quel que soit leur statut (donc un choix "paid ou all")

#21

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

Décision : Ack ainsi, on verra s'il faut ensuite trier/filtrer le listing et l'export.

#22

Mis à jour par Jean-Baptiste Jaillet il y a plus de 7 ans

  • Statut changé de En cours à Résolu (à déployer)
#23

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

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

Formats disponibles : Atom PDF