Skip to content

Comments

ADD to account_cutoff_start_end_dates the possiblity to include tax lines with start and end dates#364

Open
eLBati wants to merge 1 commit intoOCA:18.0from
eLBati:18.0-account_cutoff_start_end_dates-texes
Open

ADD to account_cutoff_start_end_dates the possiblity to include tax lines with start and end dates#364
eLBati wants to merge 1 commit intoOCA:18.0from
eLBati:18.0-account_cutoff_start_end_dates-texes

Conversation

@eLBati
Copy link
Member

@eLBati eLBati commented Feb 6, 2026

New field include_tax_lines: If enabled, tax move lines (display_type = 'tax') will also be included in the cutoff computation alongside product lines. Tax lines must have start and end dates set to be included

Rif #361

@OCA-git-bot
Copy link
Contributor

Hi @alexis-via,
some modules you are maintaining are being modified, check this out!

@eLBati eLBati force-pushed the 18.0-account_cutoff_start_end_dates-texes branch 4 times, most recently from 3042d03 to ab5c1cb Compare February 6, 2026 14:19
Copy link

@MaurizioPellegrinet MaurizioPellegrinet left a comment

Choose a reason for hiding this comment

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

Test funzionale: OK

…ines with start and end dates.

New field include_tax_lines: If enabled, tax move lines (display_type = 'tax') will also be included in the cutoff computation alongside product lines. Tax lines must have start and end dates set to be included
@eLBati eLBati force-pushed the 18.0-account_cutoff_start_end_dates-texes branch from ab5c1cb to 43c0317 Compare February 9, 2026 11:25
@eLBati
Copy link
Member Author

eLBati commented Feb 9, 2026

green tests 🍏

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.

funzionale OK

@OCA-git-bot
Copy link
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@eLBati
Copy link
Member Author

eLBati commented Feb 13, 2026

Hi @alexis-via margeable?
Thanks

@andreampiovesana
Copy link
Contributor

merge please

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants