Skip to content

[18.0][MIG] stock_unique_serial: Migration to 18.0#746

Open
deeniiz wants to merge 3 commits into18.0from
18.0-mig-stock_unique_serial
Open

[18.0][MIG] stock_unique_serial: Migration to 18.0#746
deeniiz wants to merge 3 commits into18.0from
18.0-mig-stock_unique_serial

Conversation

@deeniiz
Copy link
Copy Markdown
Collaborator

@deeniiz deeniiz commented Oct 6, 2025

No description provided.

Copilot AI review requested due to automatic review settings October 6, 2025 14:30
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 stock_unique_serial module from a previous version to Odoo 18.0. The module ensures that serial numbers can only exist in one stock location with quantity, preventing duplicate serial number assignments.

  • Migration of module to version 18.0.1.0.0
  • Addition of proper documentation structure (README, description files)
  • Implementation of validation logic for unique serial numbers in stock quants

Reviewed Changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
__manifest__.py Updated version to 18.0.1.0.0 and module metadata
__init__.py Module initialization file importing models
models/__init__.py Models initialization importing stock_quant
models/stock_quant.py Core validation logic preventing duplicate serial numbers
readme/DESCRIPTION.md Module description for documentation generation
readme/CONTRIBUTORS.md Contributors list for documentation
README.rst Generated documentation file
static/description/index.html Generated HTML documentation
pyproject.toml Build configuration file

@@ -0,0 +1,2 @@
This module ensures that the serial number have only one stock location
Copy link

Copilot AI Oct 6, 2025

Choose a reason for hiding this comment

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

Corrected grammar - 'serial number' should be plural or use 'a' article.

Suggested change
This module ensures that the serial number have only one stock location
This module ensures that serial numbers have only one stock location

Copilot uses AI. Check for mistakes.
Comment on lines +8 to +9
"summary": "This module ensures that the serial number have only one "
"stock location with quantity",
Copy link

Copilot AI Oct 6, 2025

Choose a reason for hiding this comment

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

Corrected grammar - 'serial number' should be plural or use 'a' article.

Suggested change
"summary": "This module ensures that the serial number have only one "
"stock location with quantity",
"summary": "This module ensures that serial numbers have only one "
"stock location with quantity.",

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 22, 2026
@eantones eantones added no stale and removed stale labels Mar 22, 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.

3 participants