Skip to content

[19.0][MIG] connector: Migration to 19.0#511

Open
mkoeck wants to merge 763 commits intoOCA:19.0from
mkoeck:19.0-mig-connector
Open

[19.0][MIG] connector: Migration to 19.0#511
mkoeck wants to merge 763 commits intoOCA:19.0from
mkoeck:19.0-mig-connector

Conversation

@mkoeck
Copy link

@mkoeck mkoeck commented Nov 26, 2025

Main changes

  • New security model support

    • Add a res.groups.privilege record introduced in 19.0 and adapted the existing res.groups record accordingly
  • Test suite updates for 19.0 API

    • Replace deprecated self.env2.reset() calls with self.env2.transaction.reset() in tests.
    • Adapt tests to the now read-only env.context:
      • Remove direct writes to self.env.context.
      • Use self.env() to create a new environment instead
    • Remove / replace references to demo data:
      • Demo loading order changed in 19.0, so some demo external_ids are no longer guaranteed to exist.
      • Tests now rely on explicit test data instead of demo records.

Functional impact

  • No functional feature changes are intended.
  • The goal of this PR is to make the module compatible with Odoo 19.0 while preserving existing behavior where possible.

Dependencies

This PR has been tested against the following non-merged branches of dependent modules:

Guewen Baconnier and others added 30 commits November 26, 2025 23:13
Does not do particular checks to see if each "step" is a
valid m2o field, relies on the standard and familiar
Odoo recordset errors instead.
The job function, channel and worker fields are too large for a 6 column
layout. Job function and channel make the fields overflow of the form and the
worker fields is wrapped over 3 lines
…cified (to avoid timedelta error when seconds is None) in job.py.
The job will be retried after the given number of seconds if the exception is
raised with a defined 'seconds' argument.
oca-ci and others added 19 commits November 26, 2025 23:13
Currently translated at 100.0% (10 of 10 strings)

Translation: connector-16.0/connector-16.0-connector
Translate-URL: https://translation.odoo-community.org/projects/connector-16-0/connector-16-0-connector/es/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: connector-16.0/connector-16.0-connector
Translate-URL: https://translation.odoo-community.org/projects/connector-16-0/connector-16-0-connector/
Currently translated at 100.0% (10 of 10 strings)

Translation: connector-17.0/connector-17.0-connector
Translate-URL: https://translation.odoo-community.org/projects/connector-17-0/connector-17-0-connector/it/
Currently translated at 100.0% (10 of 10 strings)

Translation: connector-17.0/connector-17.0-connector
Translate-URL: https://translation.odoo-community.org/projects/connector-17-0/connector-17-0-connector/it/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: connector-18.0/connector-18.0-connector
Translate-URL: https://translation.odoo-community.org/projects/connector-18-0/connector-18-0-connector/
@vvrossem
Copy link

vvrossem commented Feb 2, 2026

Hello @mkoeck , thank you for your contribution. Could you please attend the CI errors? 🙏

@mkoeck
Copy link
Author

mkoeck commented Feb 3, 2026

Hi @vvrossem if I remember correctly the tests are only failing because the dependant modules are not yet updated. I pulled in the open PRs for those locally and I think tests passed. I since learned that it is possible to add a temporary dependency (or whatever its called), but I am not too familiar with it. If I have the time I will look into it, but these problems should resolve once the PRs for the other modules are merged.

@imlopes
Copy link

imlopes commented Feb 23, 2026

Hello @mkoeck ,
Thank you for this work.

Could you check the failing tests and fix the pre-commit too ?
To add the dependency module (while waiting the PR get merged), you can check this commit:
OCA/report-print-send@d87a1d2

@mkoeck mkoeck force-pushed the 19.0-mig-connector branch 3 times, most recently from 9af9cfa to 1dbed57 Compare March 1, 2026 21:51
@mkoeck mkoeck force-pushed the 19.0-mig-connector branch from 5c5099f to dd03d67 Compare March 1, 2026 22:26
@mkoeck
Copy link
Author

mkoeck commented Mar 1, 2026

@imlopes I think this looks right now

@imlopes
Copy link

imlopes commented Mar 2, 2026

@mkoeck thank you for this work.
Not sure if it's mandatory (I think not) but it could be better to split your [MIG] commit in 2, keeping the test-requirements file in a separate commit.

Copy link

@imlopes imlopes 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.