Skip to content

Comments

[18.0][FIX] purchase_order_secondary_unit: revert report features requiring unmerged dependency#2953

Merged
OCA-git-bot merged 1 commit intoOCA:18.0from
qrtl:18.0-fix-purchase_order_secondary_unit
Feb 16, 2026
Merged

[18.0][FIX] purchase_order_secondary_unit: revert report features requiring unmerged dependency#2953
OCA-git-bot merged 1 commit intoOCA:18.0from
qrtl:18.0-fix-purchase_order_secondary_unit

Conversation

@yostashiro
Copy link
Member

Since many implementations are getting hit by report crashes while OCA/product-attribute#2211 has yet to be merged... 😓

Partially revert a1eea87 for purchase order and quotation report templates. The reverted portions rely on methods
(hide_secondary_uom_column, get_secondary_uom_display_mode) introduced by OCA/product-attribute#2211, which is not yet merged. Without that dependency, the reports would crash when printing.

The secondary quantity column is kept but the conditional visibility and display mode logic are removed until the dependency is available.

@qrtl

… unmerged dependency

Partially revert a1eea87 for purchase order and quotation report
templates. The reverted portions rely on methods
(hide_secondary_uom_column, get_secondary_uom_display_mode) introduced
by OCA/product-attribute#2211, which is not yet merged. Without that
dependency, the reports would crash when printing.

The secondary quantity column is kept but the conditional visibility
and display mode logic are removed until the dependency is available.
Copy link
Member

@pedrobaeza pedrobaeza left a comment

Choose a reason for hiding this comment

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

Let's fast track it for fixing the situation

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 18.0-ocabot-merge-pr-2953-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 9ef591f into OCA:18.0 Feb 16, 2026
7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at 30fbba4. 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.

3 participants