Project

General

Profile

Development #10483

Cellule générique pour les factures

Added by Serghei Mihai over 3 years ago. Updated 4 months ago.

Status:
En cours
Priority:
Normal
Assignee:
Target version:
-
Start date:
30 Mar 2016
Due date:
% Done:

0%

Patch proposed:
No
Planning:
No

Description

Avec le choix du type des factures à afficher.

0001-lingo-add-trailing-slash-to-history-invoices-endpoin.patch View (969 Bytes) Serghei Mihai, 30 Mar 2016 06:12 PM

0002-lingo-add-generic-invoice-cell-with-category-filter-.patch View (9.02 KB) Serghei Mihai, 30 Mar 2016 06:12 PM

0003-lingo-remove-ActiveItems-and-ItemsHistory-cell-10483.patch View (4.47 KB) Serghei Mihai, 30 Mar 2016 06:12 PM

0004-lingo-display-comment-for-invoice-if-present-10483.patch View (2.73 KB) Serghei Mihai, 30 Mar 2016 06:12 PM

factures_alfortville.png View (126 KB) Serghei Mihai, 30 Mar 2016 06:18 PM

0002-lingo-add-generic-invoice-cell-with-category-filter-.patch View (9.07 KB) Serghei Mihai, 31 Mar 2016 12:25 AM

0001-lingo-add-trailing-slash-to-history-invoices-endpoin.patch View (969 Bytes) Serghei Mihai, 31 Mar 2016 12:25 AM

0003-lingo-remove-ActiveItems-and-ItemsHistory-cell-10483.patch View (4.18 KB) Serghei Mihai, 31 Mar 2016 12:25 AM

0001-lingo-add-trailing-slash-to-history-invoices-endpoin.patch View (969 Bytes) Serghei Mihai, 31 Mar 2016 10:24 AM

0002-lingo-add-generic-invoice-cell-with-category-filter-.patch View (10.5 KB) Serghei Mihai, 31 Mar 2016 10:25 AM

0003-lingo-remove-ActiveItems-and-ItemsHistory-cell-10483.patch View (4.19 KB) Serghei Mihai, 31 Mar 2016 10:25 AM

0002-lingo-add-generic-invoice-cell-with-category-filter-.patch View (10.5 KB) Serghei Mihai, 31 Mar 2016 11:44 AM

0002-lingo-remove-ActiveItems-and-ItemsHistory-cell-10483.patch View (4.19 KB) Serghei Mihai, 04 Apr 2016 05:23 PM

0001-lingo-add-generic-cell-handling-all-invoice-categori.patch View (16.3 KB) Serghei Mihai, 04 Apr 2016 05:23 PM

10454

Associated revisions

Revision 58822107 (diff)
Added by Serghei Mihai over 3 years ago

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

Migrate data from existing items cell to the new one

Revision 70562bc7 (diff)
Added by Serghei Mihai over 3 years ago

lingo: remove ActiveItems and ItemsHistory cell (#10483)

Revision 31db5daa (diff)
Added by Serghei Mihai over 3 years ago

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

Use invoice no online payment reason to define line style

Revision a0148818 (diff)
Added by Serghei Mihai over 3 years ago

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

This reverts commit 31db5daa15d152965158a4a431773e215f7a7ed9.

Revision fcc6cc21 (diff)
Added by Serghei Mihai over 3 years ago

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

This reverts commit 70562bc7df964b1dc7f0e9386e252b8653998abf.

Revision 34c5ee12 (diff)
Added by Serghei Mihai over 3 years ago

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

This reverts commit 58822107c8cf548432cbadcae9cb9e17f438dfd5.

Revision f47f9b57 (diff)
Added by Serghei Mihai over 3 years ago

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

Use invoice no online payment reason to define line style

History

#1 Updated by Frédéric Péters over 3 years ago

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

#3 Updated by Serghei Mihai over 3 years ago

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 Updated by Serghei Mihai over 3 years ago

10454

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 Updated by Frédéric Péters over 3 years ago

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 Updated by Benjamin Dauvergne over 3 years ago

On l’appellerait pas InvoiceCell pour pas se tromper ?

#7 Updated by Frédéric Péters over 3 years ago

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 Updated by Frédéric Péters over 3 years ago

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 Updated by Frédéric Péters over 3 years ago

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

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

#13 Updated by Serghei Mihai over 3 years ago

Cellule revue afin de representer les types de factures sous forme de cases à cocher

#14 Updated by Frédéric Péters over 3 years ago

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

#15 Updated by Serghei Mihai over 3 years ago

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 Updated by Frédéric Péters over 3 years ago

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

#17 Updated by Serghei Mihai over 3 years ago

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

#18 Updated by Serghei Mihai over 3 years ago

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 Updated by Frédéric Péters over 3 years ago

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 Updated by Benjamin Dauvergne over 3 years ago

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 Updated by Frédéric Péters over 3 years ago

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 Updated by Frédéric Péters over 3 years ago

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 Updated by Frédéric Péters 4 months ago

  • Patch proposed changed from Yes to No

Also available in: Atom PDF