OCA / reporting-engine

Odoo Alternative Reporting Engine
https://odoo-community.org/project/tools-maintainers-30
GNU Affero General Public License v3.0
335 stars 795 forks source link

[18.0][MIG] sql_request_abstract: Migration to 18.0 #937

Closed thienvh332 closed 5 days ago

thienvh332 commented 1 month ago

Ported PRs

legalsylvain commented 1 month ago

hi. could you take a look on red CI ? thanks !

thienvh332 commented 1 month ago

Hi @legalsylvain , I updated it. Could you have a look?

OCA-git-bot commented 3 weeks ago

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

HviorForgeFlow commented 1 week ago

trivial or not, an effort to move this module to v18 has been done. Commit history will put everybody in their place. Let's merge it :)

HviorForgeFlow commented 1 week ago

/ocabot merge nobump

OCA-git-bot commented 1 week ago

On my way to merge this fine PR! Prepared branch 18.0-ocabot-merge-pr-937-by-HviorForgeFlow-bump-nobump, awaiting test results.

OCA-git-bot commented 1 week ago

@HviorForgeFlow your merge command was aborted due to failed check(s), which you can inspect on this commit of 18.0-ocabot-merge-pr-937-by-HviorForgeFlow-bump-nobump.

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.

OCA-git-bot commented 1 week ago

@HviorForgeFlow your merge command was aborted due to failed check(s), which you can inspect on this commit of 18.0-ocabot-merge-pr-937-by-HviorForgeFlow-bump-nobump.

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.

HviorForgeFlow commented 1 week ago

@thienvh332 pre-commit failed

legalsylvain commented 5 days ago

/ocabot merge nobump

OCA-git-bot commented 5 days ago

This PR looks fantastic, let's merge it! Prepared branch 18.0-ocabot-merge-pr-937-by-legalsylvain-bump-nobump, awaiting test results.

OCA-git-bot commented 5 days ago

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