Skip to content

[18.0][MIG] product_unique_barcode: Migration to 18.0#732

Open
deeniiz wants to merge 7 commits into18.0from
18.0-mig-product_unique_barcode
Open

[18.0][MIG] product_unique_barcode: Migration to 18.0#732
deeniiz wants to merge 7 commits into18.0from
18.0-mig-product_unique_barcode

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 09:31
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 product_unique_barcode module from an earlier version to Odoo 18.0, implementing unique barcode constraints for products within the same company.

  • Adds comprehensive barcode validation logic to prevent duplicate barcodes within the same company
  • Implements pre-installation hooks to check for existing barcode conflicts
  • Includes module documentation and metadata files for the new version

Reviewed Changes

Copilot reviewed 10 out of 11 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
product_unique_barcode/manifest.py Module manifest defining dependencies, version 18.0.1.0.0, and pre-init hook
product_unique_barcode/models/product.py Core logic implementing barcode uniqueness constraint with company-specific validation
product_unique_barcode/hooks.py Pre-installation hook to detect existing barcode conflicts before module installation
product_unique_barcode/init.py Module initialization importing models and hooks
product_unique_barcode/models/init.py Models package initialization
product_unique_barcode/README.rst Module documentation in reStructuredText format
product_unique_barcode/readme/DESCRIPTION.md Brief module description
product_unique_barcode/readme/CONTRIBUTORS.md Contributors information
product_unique_barcode/pyproject.toml Python project configuration
product_unique_barcode/static/description/index.html Generated HTML documentation

Comment thread product_unique_barcode/models/product.py Outdated
Comment thread product_unique_barcode/hooks.py Outdated
Comment thread product_unique_barcode/__manifest__.py Outdated
Comment thread product_unique_barcode/models/__init__.py
Comment thread product_unique_barcode/readme/DESCRIPTION.md
@deeniiz deeniiz force-pushed the 18.0-mig-product_unique_barcode branch from dec7925 to c925ac8 Compare October 1, 2025 09:40
@deeniiz deeniiz requested a review from eantones October 1, 2025 09:43
@deeniiz deeniiz force-pushed the 18.0-mig-product_unique_barcode branch from c925ac8 to 17997e8 Compare March 12, 2026 12:12
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 12, 2026

Codecov Report

❌ Patch coverage is 70.83333% with 7 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (18.0@4b294fd). Learn more about missing BASE report.

Files with missing lines Patch % Lines
product_unique_barcode/models/product.py 61.53% 5 Missing ⚠️
product_unique_barcode/hooks.py 75.00% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             18.0     #732   +/-   ##
=======================================
  Coverage        ?   70.83%           
=======================================
  Files           ?        4           
  Lines           ?       24           
  Branches        ?        4           
=======================================
  Hits            ?       17           
  Misses          ?        6           
  Partials        ?        1           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

4 participants