Skip to content

[17.0][ADD] account_invoice_report_printed_flag: add printed flag support#412

Open
atgalvez08 wants to merge 2 commits into
OCA:17.0from
BinhexTeam:17.0-add-account_invoice_report_printed_flag
Open

[17.0][ADD] account_invoice_report_printed_flag: add printed flag support#412
atgalvez08 wants to merge 2 commits into
OCA:17.0from
BinhexTeam:17.0-add-account_invoice_report_printed_flag

Conversation

@atgalvez08
Copy link
Copy Markdown

@BinhexTeam

Depends on:

description

This module extends account.move to support the printed flag
framework provided by report_printed_flag.

It adds printed report tracking capabilities to invoices,
vendor bills, and journal entries through a reusable mixin-based
integration.
Features

  • Printed flag support for account.move
  • Printed logs integration
  • Printed report names computation
  • Search filters for printed / not printed records
  • Group by printed status and printed report names
  • Multi-company support

Included components

  • account.move integration through report.printed.mixin
  • Account move views extensions
  • Search view extensions
  • Printed logs access action

Design considerations

The implementation intentionally reuses the generic framework
provided by report_printed_flag.

The module:

  • avoids duplicating framework logic
  • relies on reusable mixin inheritance
  • keeps accounting-specific logic minimal
  • remains compatible with future framework extensions

Technical details

  • Inherits from report.printed.mixin
  • Uses generic printed logs linkage through (res_model, res_id)
  • Integrates with existing account move views
  • Supports optional printed report names computation

Temporary dependency

This PR temporarily references:

through test-requirements.txt.

The temporary dependency commit will be removed once the
framework PR is merged.

Dependencies

  • account
  • report_printed_flag

Testing

Includes integration test coverage for:

  • printed flag assignment
  • printed logs creation
  • printed report names computation
  • batch printing behavior
  • non-configured reports
  • UI actions

Checklist

  • Module follows OCA structure
  • README aligned with implementation
  • Tests included
  • No duplicated framework logic
  • License headers included

@OCA-git-bot OCA-git-bot added mod:account_invoice_report_printed_flag Module account_invoice_report_printed_flag series:17.0 labels May 7, 2026
Copy link
Copy Markdown

@rrebollo rrebollo left a comment

Choose a reason for hiding this comment

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

Please consider some of my suggestions

Comment thread account_invoice_report_printed_flag/readme/TECHNICAL.md Outdated
Comment thread account_invoice_report_printed_flag/readme/TESTING.md Outdated
Comment thread account_invoice_report_printed_flag/views/account_move_views.xml
Comment thread account_invoice_report_printed_flag/views/account_move_views.xml
@atgalvez08 atgalvez08 force-pushed the 17.0-add-account_invoice_report_printed_flag branch 3 times, most recently from 2b0dc6c to 8cb299b Compare May 19, 2026 11:26
This module extends account.move to support the
report_printed_flag framework.

It adds printed report tracking capabilities to invoices,
bills, and journal entries.

Features:
- printed flag support
- printed logs access
- printed report names
- search filters and group by options
- multi-company support

The implementation relies on the reusable
report.printed.mixin model.
@atgalvez08 atgalvez08 force-pushed the 17.0-add-account_invoice_report_printed_flag branch from 8cb299b to 044c142 Compare May 19, 2026 12:17
@atgalvez08 atgalvez08 requested a review from rrebollo May 19, 2026 12:25
@atgalvez08 atgalvez08 marked this pull request as ready for review May 19, 2026 12:25
Copy link
Copy Markdown

@rrebollo rrebollo left a comment

Choose a reason for hiding this comment

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

Code Review. LGTM!

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

Labels

mod:account_invoice_report_printed_flag Module account_invoice_report_printed_flag series:17.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants