Skip to content

[REF] subscription_oca: add subscription date range to invoice#26

Open
gbrito wants to merge 1 commit intoonesteinbv:16.0from
gbrito:16.0-subscription_oca_range
Open

[REF] subscription_oca: add subscription date range to invoice#26
gbrito wants to merge 1 commit intoonesteinbv:16.0from
gbrito:16.0-subscription_oca_range

Conversation

@gbrito
Copy link

@gbrito gbrito commented Feb 28, 2025

No description provided.

@tarteo
Copy link
Member

tarteo commented Mar 21, 2025

Can you rebase your branch, please?

git fetch onestein 16.0
git rebase onestein/16.0

self.ensure_one()
if self.account_invoice_ids_count == 0:
start_date = self.date_start
end_date = self.recurring_next_date
Copy link
Member

@tarteo tarteo Apr 17, 2025

Choose a reason for hiding this comment

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

Shouldn't this be self._calculate_recurring_next_date(self.date_start)
Now the period of the first invoice is:
image
But the next is
image

I would expect the first to be 04/17/2025 and 05/17/2025

The problem could also be that recurring_next_date should be updated when date_start is changed or the template.

@gbrito gbrito requested a review from tarteo April 18, 2025 11:12
@gbrito gbrito force-pushed the 16.0-subscription_oca_range branch from a022209 to cc2701b Compare April 18, 2025 14:43
@gbrito gbrito force-pushed the 16.0-subscription_oca_range branch from cc2701b to 3d1bc60 Compare May 9, 2025 11:58
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.

2 participants