Skip to content

Conversation

@matteonext
Copy link
Contributor

@matteonext matteonext commented Oct 8, 2025

This pr allows to generate commissions based on the payment date.
For example, if you have three invoices from three different months but with a payment date in the same month, the current behavior generates three separate period, one for each month. With this change, however, all commissions are assigned to a single period.
It covers a similar scope but is managed in a different way #581

@OCA-git-bot
Copy link
Contributor

Hi @pedrobaeza,
some modules you are maintaining are being modified, check this out!

@matteonext matteonext force-pushed the 16.0-imp-commission branch 4 times, most recently from b7da3bb to f801d17 Compare October 8, 2025 14:49
@matteonext
Copy link
Contributor Author

matteonext commented Oct 8, 2025

Hi @pedrobaeza, @etobella sorry for bother!
I just opened this pr, I maybe need to add some tests and split in two commits, one for each module. What do you think about this changes, would you consider including it? Thanks a lot!

@matteonext matteonext force-pushed the 16.0-imp-commission branch 2 times, most recently from 49a3dbf to 3650074 Compare October 15, 2025 14:10
Copy link

@VBNext VBNext left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Functionality tested. LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants