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

Facturation: afficher les informations du payeur, pouvoir filtrer dessus (#75054) #32

Merged
lguerin merged 2 commits from wip/75054-invoicing-payer-info into main 2023-03-17 09:12:36 +01:00
Conversation 1 Commits 2 Files Changed 7 +231 -10
lguerin commented 2023-03-02 18:30:01 +01:00
Owner
Copy Link
No description provided.
lguerin force-pushed wip/75054-invoicing-payer-info from e6a3d52c96 to 1f95f0c02c 2023-03-03 10:50:36 +01:00 Compare
lguerin changed target branch from wip/74498-invoicing-payer-id to main 2023-03-16 10:55:18 +01:00
pmarillonnet commented 2023-03-16 16:22:33 +01:00
Owner
Copy Link

Merci pour le rebase, je commence à relire.

Merci pour le rebase, je commence à relire.
pmarillonnet approved these changes 2023-03-17 09:03:34 +01:00
pmarillonnet left a comment
Owner
Copy Link

Juste une mini remarque, histoire de ne pas ack sans rien dire. Mais c’est du détail.
Du reste c’est clair et extensivement testé, nickel, rien à redire. Ack.

Juste une mini remarque, histoire de ne pas ack sans rien dire. Mais c’est du détail. Du reste c’est clair et extensivement testé, nickel, rien à redire. Ack.
lingo/invoicing/models.py
@ -303,6 +303,10 @@ class AbstractInvoice(models.Model):
class Meta:
abstract = True
def payer_name(self):
pmarillonnet commented 2023-03-16 17:05:25 +01:00
Owner
Copy Link

Vraiment un détail, mais j’aurais mis une @property pour être raccord à celle déjà définie pour le nom d’utilisateur (AbstractInvoiceLine.user_name).

Vraiment un détail, mais j’aurais mis une `@property` pour être raccord à celle déjà définie pour le nom d’utilisateur (`AbstractInvoiceLine.user_name`).
lguerin commented 2023-03-17 09:07:54 +01:00
Author
Owner
Copy Link

fait, merci :)

fait, merci :)
lguerin added 2 commits 2023-03-17 09:07:36 +01:00
lguerin merged commit 054d63bf41 into main 2023-03-17 09:12:36 +01:00
lguerin deleted branch wip/75054-invoicing-payer-info 2023-03-17 09:12:36 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
pmarillonnet
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#32
No description provided.
Delete Branch "wip/75054-invoicing-payer-info"

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?