Skip to content

[19.0][MIG] hr_expense_tax_distribution: Migration to 19.0#351

Open
dnplkndll wants to merge 3 commits into
OCA:19.0from
ledoent:19.0-mig-hr_expense_tax_distribution
Open

[19.0][MIG] hr_expense_tax_distribution: Migration to 19.0#351
dnplkndll wants to merge 3 commits into
OCA:19.0from
ledoent:19.0-mig-hr_expense_tax_distribution

Conversation

@dnplkndll

@dnplkndll dnplkndll commented May 25, 2026

Copy link
Copy Markdown

Port of hr_expense_tax_distribution to 19.0.

Non-mechanical adaptations worth flagging

  • Sheetless Tax Distribution: Refactored the module to calculate tax splits directly on lines, overriding _prepare_receipts_vals() and _prepare_payments_vals() on hr.expense to bypass the completely removed hr.expense.sheet core model.
  • Test Modernisation: Replaced legacy sheet workflows and action calls in unit tests with direct line submission and direct posting (_post_without_wizard).
  • Standardization: Updated all translations to use the new self.env._() pattern and wrapped long lines to pass the OCA ruff checks.

@OCA-git-bot OCA-git-bot added series:19.0 mod:hr_expense_tax_distribution Module hr_expense_tax_distribution labels May 25, 2026
@dnplkndll dnplkndll force-pushed the 19.0-mig-hr_expense_tax_distribution branch from 4f05f21 to 35d1f0b Compare May 25, 2026 14:25
@dnplkndll dnplkndll changed the title [19.0][ADD] hr_expense_tax_distribution [19.0][MIG] hr_expense_tax_distribution: Migration to 19.0 May 25, 2026
@dnplkndll dnplkndll force-pushed the 19.0-mig-hr_expense_tax_distribution branch from 35d1f0b to f18cfef Compare May 25, 2026 21:00
@dnplkndll dnplkndll marked this pull request as ready for review June 3, 2026 18:24

@BhaveshHeliconia BhaveshHeliconia left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code review LGTM!

@dnplkndll dnplkndll force-pushed the 19.0-mig-hr_expense_tax_distribution branch from f18cfef to 2d2d725 Compare June 19, 2026 17:51
@dnplkndll dnplkndll force-pushed the 19.0-mig-hr_expense_tax_distribution branch from 2d2d725 to 52f4b93 Compare June 19, 2026 19:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mod:hr_expense_tax_distribution Module hr_expense_tax_distribution series:19.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants