Skip to content

[19.0][MIG] l10n_jp_summary_invoice_carryover: Migration to 19.0 - #133

Open
AungKoKoLin1997 wants to merge 5 commits into
OCA:19.0from
qrtl:19.0-mig-l10n_jp_summary_invoice_carryover
Open

[19.0][MIG] l10n_jp_summary_invoice_carryover: Migration to 19.0#133
AungKoKoLin1997 wants to merge 5 commits into
OCA:19.0from
qrtl:19.0-mig-l10n_jp_summary_invoice_carryover

Conversation

@AungKoKoLin1997

@AungKoKoLin1997 AungKoKoLin1997 commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Depends on #124

@qrtl QT6859

yostashiro and others added 5 commits June 15, 2026 03:32
…n, dependencies and report

- Carry over unpaid amounts from every previous billing in the chain,
  not just the immediately previous one. Each invoice belongs to a
  single billing, so from the third consecutive unpaid period onwards
  the outstanding balance was understated and treated as paid.
- Complete the @api.depends of _compute_show_carryover_amounts (it reads
  the partner and company show_carryover_amounts settings) and add
  company_id to the previous-billing computes (used in their domain) so
  stored values refresh when the relevant fields change.
- Use the Bootstrap 5 'text-end' class (Odoo 19) for the total billed
  amount cell so it stays right-aligned.
- Add regression tests for multi-period carryover, show_carryover_amounts
  resolution and the validate_billing freeze guard.
@OCA-git-bot OCA-git-bot added series:19.0 mod:l10n_jp_summary_invoice_carryover Module l10n_jp_summary_invoice_carryover labels Jun 15, 2026
@AungKoKoLin1997
AungKoKoLin1997 marked this pull request as ready for review June 16, 2026 09:14

@SanamiK SanamiK left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Functional review. LGTM.

Image

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

Labels

mod:l10n_jp_summary_invoice_carryover Module l10n_jp_summary_invoice_carryover series:19.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants