Skip to content

[19.0][FIX] base_report_to_printer: Only search active printers - #473

Merged
OCA-git-bot merged 1 commit into
OCA:19.0from
Studio73:19.0-fix-base_report_to_printer
Jul 30, 2026
Merged

[19.0][FIX] base_report_to_printer: Only search active printers#473
OCA-git-bot merged 1 commit into
OCA:19.0from
Studio73:19.0-fix-base_report_to_printer

Conversation

@Alexgars73

Copy link
Copy Markdown

There were cases where printers that had been taken out of service would arrive.

@OCA-git-bot OCA-git-bot added series:19.0 mod:base_report_to_printer Module base_report_to_printer labels Jul 30, 2026

@Reyes4711-S73 Reyes4711-S73 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.

LGTM

@Alexgars73

Copy link
Copy Markdown
Author

@pedrobaeza could you check that please.

@pedrobaeza pedrobaeza added this to the 19.0 milestone Jul 30, 2026
@pedrobaeza

Copy link
Copy Markdown
Member

@Alexgars73
Alexgars73 force-pushed the 19.0-fix-base_report_to_printer branch from 36ca8bf to 751eb95 Compare July 30, 2026 10:04
@Alexgars73

Copy link
Copy Markdown
Author

@pedrobaeza done!

@pedrobaeza pedrobaeza changed the title fix(report): Only search active printers [19.0][FIX] base_report_to_printer: Only search active printers Jul 30, 2026

@pedrobaeza pedrobaeza left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 19.0-ocabot-merge-pr-473-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit dff4bf1 into OCA:19.0 Jul 30, 2026
7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 7758740. Thanks a lot for contributing to OCA. ❤️

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