entrouvert
/
lingo
15
0
Fork 0
Code Issues Pull Requests 7 Releases Wiki Activity

listing des factures et de leurs lignes (#73514) #17

Merged
lguerin merged 4 commits from wip/73514-invoicing-invoice-listing into main 2023-01-27 16:01:47 +01:00
Conversation 1 Commits 4 Files Changed 11 +528 -102
lguerin commented 2023-01-19 14:43:33 +01:00
Owner
Copy Link
No description provided.
lguerin added 1 commit 2023-01-19 14:43:34 +01:00
lguerin changed title from WIP: listing des factures et de leurs lignes (#73514) to listing des factures et de leurs lignes (#73514) 2023-01-19 15:31:51 +01:00
lguerin commented 2023-01-19 15:35:30 +01:00
Author
Owner
Copy Link

0001: renommage/déplacement de vue (detail -> journal)
0002: ajout de user_name sur les models Line, pour pouvoir afficher le nom du user pour chaque ligne de facturation (pour éviter de le calculer à la volée plus tard, on le stocke au moment du calcul via les infos de subscription reçues)
0003: affichage des factures & de leurs lignes
0004: ajout de event_date sur les models Line, pour afficher la date de chaque ligne de facturation (pour éviter d'aller la chercher dans le dict event à la volée, ou dans la ligne injectée)

cf screenshot: c'est basique, et améliorable

0001: renommage/déplacement de vue (detail -> journal) 0002: ajout de user_name sur les models Line, pour pouvoir afficher le nom du user pour chaque ligne de facturation (pour éviter de le calculer à la volée plus tard, on le stocke au moment du calcul via les infos de subscription reçues) 0003: affichage des factures & de leurs lignes 0004: ajout de event_date sur les models Line, pour afficher la date de chaque ligne de facturation (pour éviter d'aller la chercher dans le dict event à la volée, ou dans la ligne injectée) cf screenshot: c'est basique, et améliorable
invoices.png
589 KiB
invoices.png
lguerin changed target branch from wip/73456-invoicing-injected-line to main 2023-01-20 16:47:46 +01:00
Ghost approved these changes 2023-01-27 14:23:01 +01:00
Ghost left a comment
First-time contributor
Copy Link

cf screenshot: c'est basique, et améliorable

Je trouve ça bien ainsi à ce stade. Ou alors tu voulais déjà que je propose des améliorations ici ? (j'en propose une minimale dans les commentaires sur les titres des factures)

J'approuve cette PR en te laissant faire les changements proposés en commentaires si tu veux.

> cf screenshot: c'est basique, et améliorable Je trouve ça bien ainsi à ce stade. Ou alors tu voulais déjà que je propose des améliorations ici ? (j'en propose une minimale dans les commentaires sur les titres des factures) J'approuve cette PR en te laissant faire les changements proposés en commentaires si tu veux.
lingo/invoicing/templates/lingo/invoicing/manager_pool_detail.html Outdated
@ -88,0 +26,4 @@
{% for line in lines %}
{% ifchanged line.invoice_id %}
{% if not forloop.first %}</ul>{% endif %}
<h3 data-invoice-id="{{ line.invoice_id }}">#{{ line.invoice_id }} {{ line.invoice.payer }} ({{ line.invoice.total_amount }})</h3>
Ghost commented 2023-01-27 14:16:54 +01:00
First-time contributor
Copy Link

Même si c'est encore draft, peut-être un tout petit poil d'explication, genre

Facture #220 adressée à Prenom Nom, total 233,00 €

parce qu'en première approx j'avais pas compris les « #202 foo:8 (116,00) » sur la copie d'écran

Mais bon, on est vraiment encore en mode draft, tu peux laisser ainsi

Même si c'est encore draft, peut-être un tout petit poil d'explication, genre Facture #220 adressée à Prenom Nom, total 233,00 € parce qu'en première approx j'avais pas compris les « #202 foo:8 (116,00) » sur la copie d'écran Mais bon, on est vraiment encore en mode draft, tu peux laisser ainsi
lguerin marked this conversation as resolved
lingo/invoicing/templates/lingo/invoicing/manager_pool_detail.html Outdated
@ -88,0 +30,4 @@
<ul class="objects-list" data-invoice-id="{{ line.invoice_id }}">
{% endifchanged %}
<li>
#{{ line.pk }} {{ line.user_name }} - {{ line.event_date|date:"d/m/Y" }} - {{ line.label }} ({{ line.total_amount }})
Ghost commented 2023-01-27 14:19:38 +01:00
First-time contributor
Copy Link

|date:"SHORT_DATE_FORMAT" pour anticiper les ventes de lingo aux US.

|date:"SHORT_DATE_FORMAT" pour anticiper les ventes de lingo aux US.
lguerin commented 2023-01-27 15:43:49 +01:00
Author
Owner
Copy Link

SHORT_DATE_FORMAT = 'j N Y'
Ça n'est pas ce que je veux afficher :)

SHORT_DATE_FORMAT = 'j N Y' Ça n'est pas ce que je veux afficher :)
lingo/invoicing/utils.py Outdated
@ -48,0 +48,4 @@
continue
user_name = '%s %s' % (subscription['user_first_name'], subscription['user_last_name'])
users[user_external_id] = (user_external_id, user_name.strip()[:250] or user_external_id)
return list(users.values())
Ghost commented 2023-01-27 14:11:01 +01:00
First-time contributor
Copy Link

micro détail, ici j'aurais juste fait
...

      users[user_external_id] = user_name.strip()[:250] or user_external_id
 return [(user_id, user_name) for user_id, user_name in users.items()]

(parce que je trouve le return plus clair ainsi sur ce qui est retourné)

micro détail, ici j'aurais juste fait ... users[user_external_id] = user_name.strip()[:250] or user_external_id return [(user_id, user_name) for user_id, user_name in users.items()] (parce que je trouve le return plus clair ainsi sur ce qui est retourné)
lguerin marked this conversation as resolved
lguerin added 3 commits 2023-01-27 15:56:36 +01:00
gitea-wip/lingo/pipeline/pr-main This commit looks good Details
cfa59aa4d1
invoicing: add event_date in line models (#73514)
lguerin merged commit cfa59aa4d1 into main 2023-01-27 16:01:47 +01:00
lguerin deleted branch wip/73514-invoicing-invoice-listing 2023-01-27 16:01:48 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
Ghost
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/lingo#17
No description provided.
Delete Branch "wip/73514-invoicing-invoice-listing"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?