Skip to content

Conversation

@ced-adhoc
Copy link

@ced-adhoc ced-adhoc commented Sep 24, 2025

Add warning for deprecated _notify_progress method in favor to _commit_progress since requires manual intervention.
More information: https://github.com/odoo/odoo/pull/207082/commits

Copilot AI review requested due to automatic review settings September 24, 2025 16:20
Copy link

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 adds migration support for the deprecated _notify_progress method being replaced with _commit_progress in Odoo 18.0 to 19.0 migration. It provides both automatic replacement for simple cases and warnings for complex patterns that require manual intervention.

  • Adds automatic text replacement rules for basic _notify_progress usage patterns
  • Includes warning messages to alert developers about deprecated method usage
  • Creates migration script structure for Odoo 18.0 to 19.0 version upgrade

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
cron_progress.yaml (text_warnings) Defines warning message for deprecated _notify_progress method usage
cron_progress.yaml (text_replaces) Provides automatic replacement patterns for simple _notify_progress cases
migrate_180_190.py Creates base migration script class for 18.0 to 19.0 migration

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@ced-adhoc ced-adhoc marked this pull request as draft September 24, 2025 17:57
@ced-adhoc ced-adhoc changed the title [ADD] Add warning and replace _notify_progress to _commit_progress [ADD] Add warning for _notify_progress to _commit_progress Sep 24, 2025
@ced-adhoc ced-adhoc changed the title [ADD] Add warning for _notify_progress to _commit_progress [ADD] Add warning for deprecate _notify_progress in favor to _commit_progress Sep 24, 2025
@ced-adhoc ced-adhoc force-pushed the 19.0-t-57439-ced-1 branch 3 times, most recently from 912e802 to 5f0b7e8 Compare October 1, 2025 15:06
@ced-adhoc ced-adhoc marked this pull request as ready for review October 1, 2025 15:12
Copy link
Contributor

@vib-adhoc vib-adhoc left a comment

Choose a reason for hiding this comment

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

LGTM

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