Skip to content

[18.0][MIG] stock_move_location_partner: Migration to 18.0#719

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

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

Conversation

@deeniiz
Copy link
Copy Markdown
Collaborator

@deeniiz deeniiz commented Sep 22, 2025

No description provided.

Copilot AI review requested due to automatic review settings September 22, 2025 16:13
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_move_location_partner module from an earlier version to Odoo 18.0, adding functionality to automatically assign partners to stock pickings during bulk stock moves based on configurable conditions.

  • Creates a complete module structure for partner assignment in stock move location wizards
  • Implements company-level configuration for bulk stock move partner assignment
  • Adds UI configuration in stock settings with multi-company support

Reviewed Changes

Copilot reviewed 14 out of 15 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
__manifest__.py Module manifest defining dependencies and data files
__init__.py Module initialization importing models and wizard
models/ Model extensions for res.company and res.config.settings
wizard/ Stock move location wizard extension with partner assignment logic
views/res_config_settings.xml UI configuration for bulk stock move partner settings
i18n/es.po Spanish translations for the module
readme/ and documentation Module documentation and description files

Comment thread stock_move_location_partner/wizard/stock_move_location_wizard.py Outdated
Comment thread stock_move_location_partner/wizard/stock_move_location_wizard.py
Comment thread stock_move_location_partner/__manifest__.py Outdated
@deeniiz deeniiz force-pushed the 18.0-mig-stock_move_location_partner branch from 117d583 to 5ad91cb Compare September 22, 2025 16:19
@deeniiz deeniiz requested a review from eantones September 22, 2025 16:19
@deeniiz deeniiz requested a review from Copilot September 22, 2025 16:19
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 14 out of 15 changed files in this pull request and generated 1 comment.

Comment thread stock_move_location_partner/wizard/stock_move_location_wizard.py
@deeniiz deeniiz force-pushed the 18.0-mig-stock_move_location_partner branch from 5ad91cb to 127f3fd Compare March 12, 2026 11:23
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 12, 2026

Codecov Report

❌ Patch coverage is 75.00000% with 6 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
...ation_partner/wizard/stock_move_location_wizard.py 45.45% 6 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             18.0     #719   +/-   ##
=======================================
  Coverage        ?   75.00%           
=======================================
  Files           ?        6           
  Lines           ?       24           
  Branches        ?        2           
=======================================
  Hits            ?       18           
  Misses          ?        6           
  Partials        ?        0           

☔ 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.

2 participants