Projet

Général

Profil

Development #10483

Cellule générique pour les factures

Ajouté par Serghei Mihai il y a environ 8 ans. Mis à jour il y a plus de 4 ans.

Statut:
En cours
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
30 mars 2016
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

Avec le choix du type des factures à afficher.


Fichiers

0001-lingo-add-trailing-slash-to-history-invoices-endpoin.patch (969 octets) 0001-lingo-add-trailing-slash-to-history-invoices-endpoin.patch Serghei Mihai, 30 mars 2016 18:12
0002-lingo-add-generic-invoice-cell-with-category-filter-.patch (9,02 ko) 0002-lingo-add-generic-invoice-cell-with-category-filter-.patch Serghei Mihai, 30 mars 2016 18:12
0003-lingo-remove-ActiveItems-and-ItemsHistory-cell-10483.patch (4,47 ko) 0003-lingo-remove-ActiveItems-and-ItemsHistory-cell-10483.patch Serghei Mihai, 30 mars 2016 18:12
0004-lingo-display-comment-for-invoice-if-present-10483.patch (2,73 ko) 0004-lingo-display-comment-for-invoice-if-present-10483.patch Serghei Mihai, 30 mars 2016 18:12
factures_alfortville.png (126 ko) factures_alfortville.png Serghei Mihai, 30 mars 2016 18:18
0002-lingo-add-generic-invoice-cell-with-category-filter-.patch (9,07 ko) 0002-lingo-add-generic-invoice-cell-with-category-filter-.patch Serghei Mihai, 31 mars 2016 00:25
0001-lingo-add-trailing-slash-to-history-invoices-endpoin.patch (969 octets) 0001-lingo-add-trailing-slash-to-history-invoices-endpoin.patch Serghei Mihai, 31 mars 2016 00:25
0003-lingo-remove-ActiveItems-and-ItemsHistory-cell-10483.patch (4,18 ko) 0003-lingo-remove-ActiveItems-and-ItemsHistory-cell-10483.patch Serghei Mihai, 31 mars 2016 00:25
0001-lingo-add-trailing-slash-to-history-invoices-endpoin.patch (969 octets) 0001-lingo-add-trailing-slash-to-history-invoices-endpoin.patch Serghei Mihai, 31 mars 2016 10:24
0002-lingo-add-generic-invoice-cell-with-category-filter-.patch (10,5 ko) 0002-lingo-add-generic-invoice-cell-with-category-filter-.patch Serghei Mihai, 31 mars 2016 10:25
0003-lingo-remove-ActiveItems-and-ItemsHistory-cell-10483.patch (4,19 ko) 0003-lingo-remove-ActiveItems-and-ItemsHistory-cell-10483.patch Serghei Mihai, 31 mars 2016 10:25
0002-lingo-add-generic-invoice-cell-with-category-filter-.patch (10,5 ko) 0002-lingo-add-generic-invoice-cell-with-category-filter-.patch Serghei Mihai, 31 mars 2016 11:44
0002-lingo-remove-ActiveItems-and-ItemsHistory-cell-10483.patch (4,19 ko) 0002-lingo-remove-ActiveItems-and-ItemsHistory-cell-10483.patch Serghei Mihai, 04 avril 2016 17:23
0001-lingo-add-generic-cell-handling-all-invoice-categori.patch (16,3 ko) 0001-lingo-add-generic-cell-handling-all-invoice-categori.patch Serghei Mihai, 04 avril 2016 17:23

Révisions associées

Révision 58822107 (diff)
Ajouté par Serghei Mihai il y a environ 8 ans

lingo: add generic cell handling all invoice categories (#10483)

Migrate data from existing items cell to the new one

Révision 70562bc7 (diff)
Ajouté par Serghei Mihai il y a environ 8 ans

lingo: remove ActiveItems and ItemsHistory cell (#10483)

Révision 31db5daa (diff)
Ajouté par Serghei Mihai il y a environ 8 ans

lingo: show invoice no online payment reason if present (#10483)

Use invoice no online payment reason to define line style

Révision a0148818 (diff)
Ajouté par Serghei Mihai il y a environ 8 ans

Revert "lingo: show invoice no online payment reason if present (#10483)"

This reverts commit 31db5daa15d152965158a4a431773e215f7a7ed9.

Révision fcc6cc21 (diff)
Ajouté par Serghei Mihai il y a environ 8 ans

Revert "lingo: remove ActiveItems and ItemsHistory cell (#10483)"

This reverts commit 70562bc7df964b1dc7f0e9386e252b8653998abf.

Révision 34c5ee12 (diff)
Ajouté par Serghei Mihai il y a environ 8 ans

Revert "lingo: add generic cell handling all invoice categories (#10483)"

This reverts commit 58822107c8cf548432cbadcae9cb9e17f438dfd5.

Révision f47f9b57 (diff)
Ajouté par Serghei Mihai il y a environ 8 ans

lingo: show invoice no online payment reason if present (#10483)

Use invoice no online payment reason to define line style

Historique

#1

Mis à jour par Frédéric Péters il y a environ 8 ans

Et plutôt que type qui "oblige" à un bien moche typ, pourquoi ne pas parler de "catégories" de factures ?

#3

Mis à jour par Serghei Mihai il y a environ 8 ans

Avec 2 commits séparés pour la migration des ActiveItems et HistoryItems vers Invoices.
Je rajoute également un patch pour placer des classes sur les lignes des factures afin de pouvoir definir des styles d'affichage particuliers.

#4

Mis à jour par Serghei Mihai il y a environ 8 ans

Le commentaire de la facture pourrait être affiché ainsi sous le libellé eventuellement avec une icone d'alerte de fontawesome.
Pour les factures impayées le div pourrait être caché.

#5

Mis à jour par Frédéric Péters il y a environ 8 ans

Laissons le 0004 à un autre ticket. Je ne suis pas favorable à l'ajout d'informations à la colonne "libellé", ça appellera discussion.

Et il faudra un autre ticket pour le litigation, aussi.

Invoice = apps.get_model('lingo', 'Invoices')

C'est plutôt confus la disparition du "s" de l'un à l'autre.

Du coup je pense que pour éviter des problème dans un avenir où on aurait un autre modèle appelé "Invoice", on devrait appeler dès maintenant la cellule "InvoicesTable".

#6

Mis à jour par Benjamin Dauvergne il y a environ 8 ans

On l’appellerait pas InvoiceCell pour pas se tromper ?

#7

Mis à jour par Frédéric Péters il y a environ 8 ans

Je m'étais mis en tête qu'on ne suffixait généralement pas avec Cell mais c'est tout le contraire, la grande majorité des modèles des cellules se terminent en Cell, donc oui, tout à fait, InvoicesCell.

#9

Mis à jour par Frédéric Péters il y a environ 8 ans

Idéalement une migration manuelle ne s'appellerait pas 0019_auto_... mais passons sur ce détail.

Le changement de nom pour la classe n'a pas été total :

  File "/home/fred/src/eo/combo/combo/public/views.py", line 96, in ajax_page_cell
    return HttpResponse(cell.render(context), content_type='text/html')
  File "/home/fred/src/eo/combo/combo/apps/lingo/models.py", line 420, in render
    return super(Invoices, self).render(context)
NameError: global name 'Invoices' is not defined

(ce qui lèverait donc la nécessité de tests unitaires…)

#11

Mis à jour par Frédéric Péters il y a environ 8 ans

Le retour en arrière ne crée que des cellules "factures actives" :

ItemsHistory = apps.get_model('lingo', 'ActiveItems')
#14

Mis à jour par Frédéric Péters il y a environ 8 ans

Je préférerais vraiment ne pas copier du code au copyright extérieur, sous une autre licence, dans combo.

#15

Mis à jour par Serghei Mihai il y a environ 8 ans

Je l'avais fait pour éviter de maintenir des packages exterieurs.
Mais ok, je package django-multiselectfield séparemment: https://dev.entrouvert.org/issues/10544

#16

Mis à jour par Frédéric Péters il y a environ 8 ans

J'imaginais plutôt fonctionner sans le code, plutôt que le taper en dépendance.

#17

Mis à jour par Serghei Mihai il y a environ 8 ans

Alors ça va être un select multiple, tout moche

#18

Mis à jour par Serghei Mihai il y a environ 8 ans

Dans l'idée de pouvoir plus tard pouvoir choisir les factures par type de paiement: prélevement, en contentieux, payable en ligne et les colonnes à afficher dans le tableau, ça me parait plus pertinent d'utiliser des cases à cocher.

#19

Mis à jour par Frédéric Péters il y a environ 8 ans

Alors ça va être un select multiple, tout moche.

Très bien, et quand on voudra autre chose on arrivera à le coder, sans dépendance externe.

#20

Mis à jour par Benjamin Dauvergne il y a environ 8 ans

Commentaire déplacé du #10494:
J'ai toujours du mal avec une section qui s'appelle "Vos factures à payer" et qui contient des trucs qu'on ne peut pas payer. Pour moi le statut de la facture devrait être donné directement dans le tableau sans click en plus, cette popup ne contient apparemment aucune donnée qu'on aurait pas déjà dans le tableau à part la raison de non-possibilité de paiement (je suppose que montant total = montant à payer dans 99,9% des cas). Je ne vois aucun lien "Payer", on est censé savoir qu'il faut cliquer sur les lignes du tableau ?

Je verrai un truc comme ça:

Vos factures            _non payées (en noir italique)_           _toutes (grisé italique)_

Référence Intitulé Date d'émission
F20150023 CANTINE MOIS D'AVRIL 2016
payable en ligne avant le 20 mai 2016
25 avril 2016 [ Payer ] [ Télécharger]
F20150023 PÉRISCOLAIRE AVRIL 2016
payable en ligne avant le 22 mai 2016
27 avril 2016 [ Payer ] [ Télécharger]
F20150023 PÉRISCOLAIRE MARS 2016
en contentieux, traitement en cours.
27 avril 2016 [ Télécharger]
F20150023 CANTINE MARS 2016
sera prélevée le 29 avril 2016
27 avril 2016 [ Télécharger]

Je passe que si une facture est en contentieux on veut le savoir tout de suite, même si on ne peut pas la payer, et pour celle en prélèvement on veut savoir quand ce sera prélevé.

Si on clique sur "toutes":

Vos factures            _à payer (grisé italique)_           _toutes (en noir italique)_
Référence Intitulé Date d'émission
F20150023 CANTINE MOIS D'AVRIL 2016
payable en ligne avant le 20 mai 2016
25 avril 2016 [ Payer ]
[ Télécharger]
F20150023 PÉRISCOLAIRE AVRIL 2016
payable en ligne avant le 22 mai 2016
27 avril 2016 [ Payer ]
[ Télécharger]
F20150023 PÉRISCOLAIRE MARS 2016
en contentieux, traitement en cours.
27 avril 2016 [ Télécharger]
F20150023 CANTINE MARS 2016
sera prélevée le 29 avril 2016
27 avril 2016 [ Télécharger]
F20150023 PÉRISCOLAIRE FÉVRIER 2016
prélevée le 9 avril 2016
27 avril 2016 [ Télécharger]
F20150023 PÉRISCOLAIRE FÉVRIER 2016
payée en ligne le 9 avril 2016
27 avril 2016 [ Télécharger]

[_Voir les factures précédentes_]

Si on clique sur voir les factures précédentes on charge d'autres factures en AJAX par paquet de 3 mois, si on a une facture PDF, télécharger donne tous les détails pas besoin de bouton [Voir] si on a pas de PDF je propose de balancer un HTML pseudo-"imprimable" genre facturette CB qu'on aurait mis avant dans la popup [Voir]. Mais [Voir] + [Télécharger] c'est trop.

#21

Mis à jour par Frédéric Péters il y a environ 8 ans

Je viens de créer https://dev.entrouvert.org/projects/publik/wiki/Mockups_Publik_Jarticule_2016 , sans commentaires pour le moment, mais il y a des bouts qui sont faits et te rejoignent (voir + télécharger, c'est trop, on est tous d'accord). Aussi, la cellule unique avec filtre en haut, c'est quelque chose qu'on trouvait dans les mockup Orléans.

#22

Mis à jour par Frédéric Péters il y a environ 8 ans

J'écrivais :

Alors ça va être un select multiple, tout moche.

Très bien, et quand on voudra autre chose on arrivera à le coder, sans dépendance externe.

Aussi, il semblerait possible que Django intègre de base select2 dans une version à venir et j'ai vu à cette occasion qu'il gérait aussi les <select multi>

#23

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

  • Patch proposed changé de Oui à Non

Formats disponibles : Atom PDF