Skip to content

[18.0][MIG] partner_review: Migration to 18.0#706

Open
deeniiz wants to merge 5 commits into18.0from
18.0-mig-partner_review
Open

[18.0][MIG] partner_review: Migration to 18.0#706
deeniiz wants to merge 5 commits into18.0from
18.0-mig-partner_review

Conversation

@deeniiz
Copy link
Copy Markdown
Collaborator

@deeniiz deeniiz commented Sep 19, 2025

No description provided.

Copilot AI review requested due to automatic review settings September 19, 2025 07:01
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 partner_review module from an earlier Odoo version to 18.0, adding a "To review" boolean field to partner records with corresponding view modifications and filters.

Key changes include:

  • Addition of a new boolean field to_review to the res.partner model
  • Implementation of view extensions to display the field and provide filtering capabilities
  • Creation of standard module structure with proper documentation and translations

Reviewed Changes

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

Show a summary per file
File Description
partner_review/__manifest__.py Module manifest defining version 18.0.1.0.0 and dependencies
partner_review/models/res_partner.py Extends res.partner model with to_review boolean field
partner_review/views/res_partner_view.xml Adds form field and search filters for the to_review functionality
partner_review/i18n/es.po Spanish translation file with outdated version reference
partner_review/README.rst Module documentation with project information
partner_review/readme/CONTRIBUTORS.md Contributors list in markdown format
partner_review/static/description/index.html Auto-generated HTML documentation
partner_review/models/__init__.py Model imports initialization
partner_review/__init__.py Module initialization
partner_review/pyproject.toml Build system configuration

Comment thread partner_review/i18n/es.po Outdated
Comment thread partner_review/__manifest__.py Outdated
@deeniiz deeniiz force-pushed the 18.0-mig-partner_review branch from 2c4f4b2 to d1c8552 Compare September 19, 2025 07:04
@deeniiz deeniiz requested a review from eantones September 19, 2025 07:06
@deeniiz deeniiz requested a review from Copilot September 19, 2025 07:08
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 10 out of 12 changed files in this pull request and generated 2 comments.

Comment thread partner_review/readme/CONTRIBUTORS.md
Comment thread partner_review/i18n/es.po
@deeniiz deeniiz force-pushed the 18.0-mig-partner_review branch from d1c8552 to f7ca8ef Compare September 19, 2025 07:18
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 1, 2026

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 1, 2026
@eantones eantones added no stale and removed stale labels Mar 1, 2026
@deeniiz deeniiz force-pushed the 18.0-mig-partner_review branch from f7ca8ef to 76985ef Compare March 12, 2026 10:35
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 12, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (18.0@4b294fd). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff            @@
##             18.0      #706   +/-   ##
========================================
  Coverage        ?   100.00%           
========================================
  Files           ?         3           
  Lines           ?         6           
  Branches        ?         0           
========================================
  Hits            ?         6           
  Misses          ?         0           
  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.

4 participants