Skip to content

[IMP] server_environment: module uninstallation#274

Merged
OCA-git-bot merged 1 commit into
OCA:19.0from
camptocamp:19.0-server-env-module-uninstall
Jul 1, 2026
Merged

[IMP] server_environment: module uninstallation#274
OCA-git-bot merged 1 commit into
OCA:19.0from
camptocamp:19.0-server-env-module-uninstall

Conversation

@gurneyalex

Copy link
Copy Markdown
Member

Add a helper to manage the restoring of the database columns when a module using server_environment is uninstalled or the dependency on server_environment is dropped.

Document how to use the helper in an uninstall script or in an upgrade script (if a new version of the addon drops the dependency).

This is a forward port of #261

@OCA-git-bot OCA-git-bot added mod:server_environment Module server_environment series:19.0 labels Jun 29, 2026
Add a helper to manage the restoring of the database columns when a
module using `server_environment` is uninstalled or the dependency on
`server_environment` is dropped.

Document how to use the helper in an uninstall script or in an upgrade
script (if a new version of the addon drops the dependency).
@ivantodorovich ivantodorovich force-pushed the 19.0-server-env-module-uninstall branch from 0d05d23 to b89ebd9 Compare June 30, 2026 19:08
@ivantodorovich

Copy link
Copy Markdown
Contributor

The CI was failing with:

errors that caused failure (1):
2026-06-29 07:18:40,585 307 WARNING odoo odoo.addons.server_environment.uninstall: restore_env_managed_columns: field '__nonexistent_field_xyz__' not found on res.partner, skipping 

Error: Errors detected in log.

I updated your PR to fix it with self.assertLogs: https://github.com/OCA/server-env/compare/0d05d23acd6de3edfaf678d9f0fe895e4af3397e..b89ebd9d761f535f6fba2af0204a6faf8aff3f97

@etobella

etobella commented Jul 1, 2026

Copy link
Copy Markdown
Member

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 19.0-ocabot-merge-pr-274-by-etobella-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit a954ee5 into OCA:19.0 Jul 1, 2026
5 of 7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at 34b9185. Thanks a lot for contributing to OCA. ❤️

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.

5 participants