Development #10483
Cellule générique pour les factures
0%
Description
Avec le choix du type des factures à afficher.
Fichiers
Révisions associées
lingo: remove ActiveItems and ItemsHistory cell (#10483)
lingo: show invoice no online payment reason if present (#10483)
Use invoice no online payment reason to define line style
Revert "lingo: show invoice no online payment reason if present (#10483)"
This reverts commit 31db5daa15d152965158a4a431773e215f7a7ed9.
Revert "lingo: remove ActiveItems and ItemsHistory cell (#10483)"
This reverts commit 70562bc7df964b1dc7f0e9386e252b8653998abf.
Revert "lingo: add generic cell handling all invoice categories (#10483)"
This reverts commit 58822107c8cf548432cbadcae9cb9e17f438dfd5.
lingo: show invoice no online payment reason if present (#10483)
Use invoice no online payment reason to define line style
Historique
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 ?
Mis à jour par Serghei Mihai il y a environ 8 ans
- Fichier 0001-lingo-add-trailing-slash-to-history-invoices-endpoin.patch 0001-lingo-add-trailing-slash-to-history-invoices-endpoin.patch ajouté
- Fichier 0002-lingo-add-generic-invoice-cell-with-category-filter-.patch 0002-lingo-add-generic-invoice-cell-with-category-filter-.patch ajouté
- Fichier 0003-lingo-remove-ActiveItems-and-ItemsHistory-cell-10483.patch 0003-lingo-remove-ActiveItems-and-ItemsHistory-cell-10483.patch ajouté
- Fichier 0004-lingo-display-comment-for-invoice-if-present-10483.patch 0004-lingo-display-comment-for-invoice-if-present-10483.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
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.
Mis à jour par Serghei Mihai il y a environ 8 ans
- Fichier factures_alfortville.png factures_alfortville.png ajouté
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é.
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".
Mis à jour par Benjamin Dauvergne il y a environ 8 ans
On l’appellerait pas InvoiceCell pour pas se tromper ?
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.
Mis à jour par Serghei Mihai il y a environ 8 ans
- Fichier 0002-lingo-add-generic-invoice-cell-with-category-filter-.patch 0002-lingo-add-generic-invoice-cell-with-category-filter-.patch ajouté
- Fichier 0001-lingo-add-trailing-slash-to-history-invoices-endpoin.patch 0001-lingo-add-trailing-slash-to-history-invoices-endpoin.patch ajouté
- Fichier 0003-lingo-remove-ActiveItems-and-ItemsHistory-cell-10483.patch 0003-lingo-remove-ActiveItems-and-ItemsHistory-cell-10483.patch ajouté
Cellule renommée en InvoicesCell
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…)
Mis à jour par Serghei Mihai il y a environ 8 ans
- Fichier 0001-lingo-add-trailing-slash-to-history-invoices-endpoin.patch 0001-lingo-add-trailing-slash-to-history-invoices-endpoin.patch ajouté
- Fichier 0002-lingo-add-generic-invoice-cell-with-category-filter-.patch 0002-lingo-add-generic-invoice-cell-with-category-filter-.patch ajouté
- Fichier 0003-lingo-remove-ActiveItems-and-ItemsHistory-cell-10483.patch 0003-lingo-remove-ActiveItems-and-ItemsHistory-cell-10483.patch ajouté
Corrigé, avec un test unitaire minimaliste de rendu d'une cellule
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')
Mis à jour par Serghei Mihai il y a environ 8 ans
- Fichier 0002-lingo-add-generic-invoice-cell-with-category-filter-.patch 0002-lingo-add-generic-invoice-cell-with-category-filter-.patch ajouté
Bien vu.
Mis à jour par Serghei Mihai il y a environ 8 ans
- Fichier 0001-lingo-add-generic-cell-handling-all-invoice-categori.patch 0001-lingo-add-generic-cell-handling-all-invoice-categori.patch ajouté
- Fichier 0002-lingo-remove-ActiveItems-and-ItemsHistory-cell-10483.patch 0002-lingo-remove-ActiveItems-and-ItemsHistory-cell-10483.patch ajouté
Cellule revue afin de representer les types de factures sous forme de cases à cocher
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.
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
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.
Mis à jour par Serghei Mihai il y a environ 8 ans
Alors ça va être un select multiple, tout moche
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.
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.
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.
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.
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>
lingo: add generic cell handling all invoice categories (#10483)
Migrate data from existing items cell to the new one