Skip to content

[18.0][MIG] purchase_order_new_line_uom: Migration to 18.0#734

Open
deeniiz wants to merge 6 commits into18.0from
18.0-mig-purchase_order_new_line_uom
Open

[18.0][MIG] purchase_order_new_line_uom: Migration to 18.0#734
deeniiz wants to merge 6 commits into18.0from
18.0-mig-purchase_order_new_line_uom

Conversation

@deeniiz
Copy link
Copy Markdown
Collaborator

@deeniiz deeniiz commented Oct 1, 2025

No description provided.

Copilot AI review requested due to automatic review settings October 1, 2025 14:18
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR migrates the purchase_order_new_line_uom module from a previous version to Odoo 18.0. The module allows editing the unit of measure (UOM) on new purchase order lines that haven't been linked to any invoices or stock moves.

  • Migration of module structure and files to 18.0 compatibility
  • Addition of copyright headers and standardized file structure
  • Implementation of view inheritance to control UOM field editability

Reviewed Changes

Copilot reviewed 7 out of 9 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
views/purchase_views.xml Implements view inheritance to make UOM field readonly when purchase order lines have invoices or moves
manifest.py Module manifest with dependencies and metadata for 18.0
README.rst Module documentation with description and contributor information
readme/DESCRIPTION.md Brief module description for documentation generation
readme/CONTRIBUTORS.md List of module contributors
pyproject.toml Python project configuration file
static/description/index.html Generated HTML documentation file

Comment thread purchase_order_new_line_uom/readme/DESCRIPTION.md Outdated
Comment thread purchase_order_new_line_uom/README.rst Outdated
Comment thread purchase_order_new_line_uom/static/description/index.html Outdated
@deeniiz deeniiz force-pushed the 18.0-mig-purchase_order_new_line_uom branch from df2794e to c209de2 Compare October 1, 2025 14:21
@deeniiz deeniiz requested review from Copilot and eantones October 1, 2025 14:22
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 7 out of 9 changed files in this pull request and generated 1 comment.

Comment on lines +15 to +19
<field
name="invoice_lines"
column_invisible="1"
widget="many2many_tags"
/>
Copy link

Copilot AI Oct 1, 2025

Choose a reason for hiding this comment

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

Duplicate field definition for 'invoice_lines'. This field is being added after finding the existing 'invoice_lines' field in the xpath, creating a duplicate field in the view which could cause rendering issues or unexpected behavior.

Suggested change
<field
name="invoice_lines"
column_invisible="1"
widget="many2many_tags"
/>

Copilot uses AI. Check for mistakes.
@github-actions
Copy link
Copy Markdown

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions Bot added the stale label Mar 15, 2026
@eantones eantones added no stale and removed stale labels Mar 16, 2026
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