search

OCA / server-tools

Tools for Odoo Administrators to improve some technical features on Odoo.
https://odoo-community.org/psc-teams/tools-30
GNU Affero General Public License v3.0
705 stars 1.5k forks source link

[17.0][FW][FIX] database_cleanup: Bug introduced by #2098 #2988

Closed aleuffre closed 2 months ago

aleuffre commented 4 months ago

https://github.com/OCA/server-tools/commit/0e3a449471fac29d8eb946a3e4a7b97070a0137b

pedrobaeza commented 4 months ago

Please use proper commit message: https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#commit-message

aleuffre commented 4 months ago

Thank you. Updated the commit message.

Failing tests are unrelated.

OCA-git-bot commented 3 months ago

This PR looks fantastic, let's merge it! Prepared branch 17.0-ocabot-merge-pr-2988-by-pedrobaeza-bump-patch, awaiting test results.

OCA-git-bot commented 3 months ago

@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 17.0-ocabot-merge-pr-2988-by-pedrobaeza-bump-patch.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

pedrobaeza commented 3 months ago

Please check CI

StefanRijnhart commented 2 months ago

Looks like https://github.com/OCA/server-tools/pull/3020 fixes some issues with the test of tracking_manager. Let's try again.

/ocabot merge patch

OCA-git-bot commented 2 months ago

This PR looks fantastic, let's merge it! Prepared branch 17.0-ocabot-merge-pr-2988-by-StefanRijnhart-bump-patch, awaiting test results.

OCA-git-bot commented 2 months ago

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