Skip to content

[14.0][ADD] l10n_pt_stock_invoicexpress_vehicle_daily: Glue module to integ…#737

Merged
eantones merged 1 commit into14.0from
14.0-add-l10n_pt_stock_invoiceexpress_vehicle_daily
Oct 1, 2025
Merged

[14.0][ADD] l10n_pt_stock_invoicexpress_vehicle_daily: Glue module to integ…#737
eantones merged 1 commit into14.0from
14.0-add-l10n_pt_stock_invoiceexpress_vehicle_daily

Conversation

@eantones
Copy link
Copy Markdown
Member

@eantones eantones commented Oct 1, 2025

…rate InvoiceXpress transport documents with vehicle daily stock tracking

…rate InvoiceXpress transport documents with vehicle daily stock tracking
Copilot AI review requested due to automatic review settings October 1, 2025 16:55
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

Adds a new auto-install Odoo bridge module integrating Portuguese InvoiceXpress transport documents with the vehicle daily stock tracking module to synchronize and manage license plate data.

  • Introduces a stored computed license_plate field on stock.picking sourced from the location.
  • Extends the stock move location wizard to propagate the Portuguese license plate to created pickings.
  • Hides a redundant license plate field in the picking form when both dependent modules are installed.

Reviewed Changes

Copilot reviewed 13 out of 14 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
setup/l10n_pt_stock_invoicexpress_vehicle_daily/setup.py Adds packaging setup for the new addon.
setup/.../l10n_pt_stock_invoicexpress_vehicle_daily Symlink path reference for addon packaging.
l10n_pt_stock_invoicexpress_vehicle_daily/manifest.py Declares new bridge module metadata and dependencies.
l10n_pt_stock_invoicexpress_vehicle_daily/models/stock_picking.py Adds computed/stored license_plate field inheriting stock.picking.
l10n_pt_stock_invoicexpress_vehicle_daily/wizard/stock_move_location.py Extends wizard to write license plate on created picking.
l10n_pt_stock_invoicexpress_vehicle_daily/views/stock_picking_views.xml Hides redundant license plate field in UI.
l10n_pt_stock_invoicexpress_vehicle_daily/models/init.py Loads model extension.
l10n_pt_stock_invoicexpress_vehicle_daily/wizard/init.py Loads wizard extension.
l10n_pt_stock_invoicexpress_vehicle_daily/README.rst Generated README describing the new module.
l10n_pt_stock_invoicexpress_vehicle_daily/readme/DESCRIPTION.rst Module description consistent with manifest summary.
l10n_pt_stock_invoicexpress_vehicle_daily/readme/CONTRIBUTORS.rst Lists contributors.
l10n_pt_stock_invoicexpress_vehicle_daily/static/description/index.html Generated HTML description for Apps view.
l10n_pt_stock_invoicexpress_vehicle_daily/views/stock_picking_views.xml View inheritance for hiding field.

Comment thread l10n_pt_stock_invoicexpress_vehicle_daily/models/stock_picking.py
@@ -0,0 +1,21 @@
# Copyright 2025 NuoBiT Solutions - Eric Antones <eantones@nuobit.com>
# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).

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.

Insert an additional blank line so there are exactly 2 blank lines between the license header and the start of the code block as required by the coding standard.

Suggested change

Copilot uses AI. Check for mistakes.
Comment on lines +6 to +7
"summary": "Bridge module integrating InvoiceXpress transport "
"documents with vehicle daily stock tracking for Portuguese Tax Authorities",
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.

Multi-line implicit string concatenation should be wrapped in parentheses per the guideline; rewrite as ("Bridge module integrating ... Authorities") or place the full text on a single line.

Suggested change
"summary": "Bridge module integrating InvoiceXpress transport "
"documents with vehicle daily stock tracking for Portuguese Tax Authorities",
"summary": (
"Bridge module integrating InvoiceXpress transport "
"documents with vehicle daily stock tracking for Portuguese Tax Authorities"
),

Copilot uses AI. Check for mistakes.
Comment thread l10n_pt_stock_invoicexpress_vehicle_daily/README.rst
Comment thread l10n_pt_stock_invoicexpress_vehicle_daily/models/stock_picking.py
@eantones eantones merged commit 1e23dfc into 14.0 Oct 1, 2025
11 checks passed
@eantones eantones deleted the 14.0-add-l10n_pt_stock_invoiceexpress_vehicle_daily branch October 1, 2025 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants